search

curiousdannii-testing / inform7-imported-bugs

0 stars 0 forks source link

[I7-1883] [Mantis 1919] Testing commands holding multiple objects won't compile #201

Closed curiousdannii-testing closed 2 years ago

curiousdannii-testing commented 2 years ago

Reported by : Angstsmurf

Description :

In 6G60, you could write test commands holding an unlimited number of objects in a specific location. In more recent versions, you seem to be limited to holding two objects and no given location or one object and one given location.

++ 15% (Drawing inferences)
>--> You wrote 'Test me with "z / z" in Example location holding the cate and
the armband' (source text, line 7): but I didn't recognise the requirements
for this test scenario, which should be 'test ... with ... in ...' or '...
holding ...'
Inform 7 has finished.
>--> You wrote 'Test me with "z / z" in Example location holding the cate and
the armband' (source text, line 7): again, I didn't recognise the
requirements for this test scenario.
++ Ended: Translation failed: 2 problems found

Compiler finished with code 1

Steps to reproduce :

"Test Case" by Angstsmurf

Example Location is a room. 

The cate is a thing. The armband is a thing.

Test me with "z / z" in Example location holding the cate and the armband.

Additional information :

imported from: [Mantis 1919] Testing commands holding multiple objects won't compile
  • status: Closed
  • resolution: Resolved
  • resolved: 2022-04-13T12:45:11+10:00
  • imported: 2022/01/10
curiousdannii-testing commented 2 years ago

557058:4c095ffd-6d6f-47ce-9e73-77c613347b86:

Comment by zarf :
Confirmed.

You can work around this by including "PURLOIN OBJ" in the list of test commands.

curiousdannii-testing commented 2 years ago

557058:4c095ffd-6d6f-47ce-9e73-77c613347b86:

Comment by Angstsmurf :
At least in Counterfeit Monkey, purloin tends to get stuck in disambiguation. So I ended up re-implementing the test command.

curiousdannii-testing commented 2 years ago

61eedb62875fc10070240916:

Fixed via this commit: https://github.com/ganelson/inform/commit/1181c19c9bd23d49c0282dcca8e5075c7c26b795

Comment by Graham Nelson:
This was absolutely a bug, and there was no intention to withdraw the ability to have more than two clauses after 6G60.