stevedlawrence commented on code in PR #1579:
URL: https://github.com/apache/daffodil/pull/1579#discussion_r2478872180
##########
daffodil-test-integration/src/test/scala/org/apache/daffodil/cliTest/TestCLIDebugger.scala:
##########
@@ -1208,10 +1207,6 @@ class TestCLIDebugger {
cli.sendLine("step")
cli.sendLine("step")
cli.sendLine("step")
- cli.expect(regexp("RegionSplit.* for cell"))
- cli.sendLine("info suspensions")
- cli.expect(regexp("Suppressable.* for cell"))
- cli.expect(regexp("RegionSplit.* for cell"))
Review Comment:
Removing these `expects` and `sendLines` changes this test. And looking at
the commit when this was added, it was specifically about checking to make sure
`info suspensions` and `info diff` with suspensions works as expected. So I'm
not sure this is the right fix.
If this schema doesn't create suspensions anymore then we should probably
switch this test to use a different schema so that we can keep info suspensions
and info diff with suspensions in the test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
