Open 0pdd opened 1 month ago
@0pdd thanks for the report, here is a feedback:
Please fix the bug report in order it to get resolved faster.
Analyzed with gpt-4
@0pdd thanks for the report, here is a feedback: * The todo message is not clear about the location ...
I see you're talking to me, but I can't reply since I'm not a chat bot.
@maxonfjvipon me
@yegor256 let's transfer it to xax repo. I would like to create alternative yaml test cases:
# yaml
---
xsls:
- /org/eolang/maven/pre/align-test-classes.xsl
before:
<Some ML>
after:
<Another xml>
Such tests would extend documentation of xsl's.
@maxonfjvipon please, help here
@maxonfjvipon Can you transfer?
@levBagryansky why do you want to create xmir2xmir test?
@maxonfjvipon I believe that code examples in comments are bad decision (they lose relevance quickly). It is much better to document them via tests.
@levBagryansky comparing XML just by simple text matching is a bad idea because of a lot of reasons, for example attributes can switch places and so on. That's why we use XPATH and that's the main purpose of XAX repo
@maxonfjvipon It's definitely better than writing input/output in the comments. Updating xsl that has such test, programmer will update it's test. But perhaps the corresponding infra for test should be in other place.
@levBagryansky comments with documentation in XSL are desirable but not mandatory. You're right - the main "documentation" is tests. But we already have mechanism of XML validation - XPATH, we also have infrastructure for it in eo-runtime
.
@maxonfjvipon We write such input/output in the comments. Thus, xax does not fit for such cases
@levBagryansky as I said - input/output is not mandatory, you should write at least "something" so next user understand what the transformation does
@yegor256 please help
The puzzle
1249-cbc24ee5
from #1249 has to be resolved:https://github.com/objectionary/eo/blob/f26ee33ccd344724a794446e34a5ae555d2e573b/eo-maven-plugin/src/main/resources/org/eolang/maven/pre/remove-high-level-inner-classes.xsl#L27-L28
The puzzle was created by Yegor Bugayenko on 15-Jul-24.
Estimate: 90 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be \"done\" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.