Skip to content

Commit e586092

Browse files
committed
Update e2etest notes again
1 parent 65b8879 commit e586092

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

e2etest/zz_tests_to_add.go

+4
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,10 @@ package e2etest
2323
// Need to hook it into build
2424
// Need to gradually remove python tests, but only remove those that are covered by the new suite.
2525
// Also need to remove the "unit" tests in cmd that actually hit the service. Probably they can all be covered by the new suite.
26+
// What do we want to do about the recursive flag? Just leave it how it is now, set to a specific value for each test?
27+
// Or, do we want to have a feature where we can tell the framework to also run additional non-recursive scenarios? We shouldn't have to write any more
28+
// test code, because the framework could just automatically know "Oh, I'm doing the non-recursive case. Anything in "shouldTransfer" which
29+
// is not at root level should be treated as if its in "shouldIgnore".
2630
//
2731
// Framework gaps
2832
// IMPORTANT Creating remote files more quickly (or at least in parallel). Right now, it takes too long to do the setup for tests with non-trivial file sizes

0 commit comments

Comments
 (0)