Skip to content

Commit d4fb22c

Browse files
committed
tests: Update blocking tests
1 parent 211dd64 commit d4fb22c

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

tests/test_dimension.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2044,21 +2044,21 @@ def test_topofusion_w_subdims_conddims(self):
20442044

20452045
# Check generated code -- expect the gsave equation to be scheduled together
20462046
# in the same loop nest with the fsave equation
2047-
bns, _ = assert_blocking(op, {'x0_blk0', 'x2_blk0', 'x1_blk0'})
2047+
bns, _ = assert_blocking(op, {'x0_blk0', 'x1_blk0', 'x2_blk0'})
20482048
exprs = FindNodes(Expression).visit(bns['x0_blk0'])
20492049
assert len(exprs) == 2
20502050
assert exprs[0].write is f
20512051
assert exprs[1].write is g
20522052

2053+
exprs = FindNodes(Expression).visit(bns['x1_blk0'])
2054+
assert len(exprs) == 1
2055+
assert exprs[0].write is h
2056+
20532057
exprs = FindNodes(Expression).visit(bns['x2_blk0'])
20542058
assert len(exprs) == 2
20552059
assert exprs[0].write is fsave
20562060
assert exprs[1].write is gsave
20572061

2058-
exprs = FindNodes(Expression).visit(bns['x1_blk0'])
2059-
assert len(exprs) == 1
2060-
assert exprs[0].write is h
2061-
20622062
def test_topofusion_w_subdims_conddims_v2(self):
20632063
"""
20642064
Like `test_topofusion_w_subdims_conddims` but with more SubDomains,

tests/test_subdomains.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -680,8 +680,8 @@ class Dummy(SubDomainSet):
680680
# Make sure it jit-compiles
681681
op.cfunction
682682

683-
assert_structure(op, ['t,n0', 't,n0,ix0_blk0,iy0_blk0,x,y,z'],
684-
't,n0,ix0_blk0,iy0_blk0,x,y,z')
683+
assert_structure(op, ['t,n0', 't,n0,x0_blk0,y0_blk0,x,y,z'],
684+
't,n0,x0_blk0,y0_blk0,x,y,z')
685685

686686
# Drag a rebuilt MultiSubDimension out of the operator
687687
dims = {d.name: d for d in FindSymbols('dimensions').visit(op)}

0 commit comments

Comments
 (0)