Move citation-number test that depend on off-spec behavior to off-spec folder
#64
Conversation
These test were modified in the past so they no longer match their names
|
For some odd reason (given the fairly infrequent merges here), you have merge conflicts! Aside from fixing those, RTM? |
|
@bdarcus are you still seeing conflicts? In the GitHub interface, I’m seeing |
|
Oh wow; maybe a GitHub issue then. Here's what I see:
|
|
Ah, one thing I just noticed … In the future, I need to PR from branches on my fork. I made my test-suite fork a submodule of my Swift processor project so whatever I see to adjust there is immediately incorporated into any open PR here — not quite what I intended. It also makes the commit description weird since I’m generally describing processor changes, not the test tweaks. |
|
@bdarcus Maybe I’ll try to clean this up by closing this PR, making a branch on my fork, as just described, and PR’ing from there 🤷♂️ |
|
Made new PR from branch #65 |
Per conversation in #53 (comment)
Tests that show a reversed
citation-numberdepend on overloaded behavior not defined in the specification. I left in placecitation-numbertests that show an ascendingcitation-number. I think those are basically no-ops anyway.I considered updating
processor.pywith an argument to optionally include this folder but I’m not sure how often it’s used. It didn’t look like it was updated for theprocessor-tests/humans/experimentsfolder so maybe not needed.