Skip to content

Commit 837a156

Browse files
authored
[Test] Remove case-of-case from scoping tests (#7175)
Fixes #7170 by simply removing the test.
1 parent f017be4 commit 837a156

File tree

1 file changed

+3
-8
lines changed
  • plutus-core/untyped-plutus-core/testlib/Scoping

1 file changed

+3
-8
lines changed

plutus-core/untyped-plutus-core/testlib/Scoping/Spec.hs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import UntypedPlutusCore
88
import UntypedPlutusCore.Generators.Hedgehog.AST (genProgram, genTerm, mangleNames, runAstGen)
99
import UntypedPlutusCore.Mark
1010
import UntypedPlutusCore.Rename.Internal
11-
import UntypedPlutusCore.Transform.CaseOfCase (caseOfCase)
1211
import UntypedPlutusCore.Transform.CaseReduce (caseReduce)
1312
import UntypedPlutusCore.Transform.Cse (cse)
1413
import UntypedPlutusCore.Transform.FloatDelay (floatDelay)
@@ -70,13 +69,9 @@ test_names = testGroup "names"
7069
rename
7170
, T.test_scopingSpoilRenamer (genProgram @DefaultFun) markNonFreshProgram
7271
renameProgramM
73-
, T.test_scopingGood "case-of-case" (genTerm @DefaultFun) T.BindingRemovalOk T.PrerenameYes $
74-
evalSimplifierT . caseOfCase
75-
, -- COKC removes entire branches, some of which are going to contain binders, but we still use
76-
-- 'BindingRemovalNotOk', because the 'EstablishScoping' instance does not attempt to
77-
-- reference bindings from one branch in another one. We could do that, but then we'd be
78-
-- removing not just TODO.
79-
T.test_scopingGood "case-of-known-constructor"
72+
-- We don't test case-of-case, because it duplicates binders and we don't support that in the
73+
-- scoping tests machinery.
74+
, T.test_scopingGood "case-of-known-constructor"
8075
(genTerm @DefaultFun)
8176
T.BindingRemovalOk -- COKC removes branches, which may (and likely do) contain bindings.
8277
T.PrerenameYes

0 commit comments

Comments
 (0)