Skip to content

Conversation

@k1o0
Copy link
Contributor

@k1o0 k1o0 commented Jul 19, 2019

No description provided.

@k1o0 k1o0 requested a review from jkbhagatio July 19, 2019 08:56
@todo
Copy link

todo bot commented Jul 19, 2019

Use DELETE to test both creating new session and viewing

% TODO Use DELETE to test both creating new session and viewing
% existing
p = testCase.Panel;
baseURL = p.AlyxInstance.BaseURL;
testCase.Mock.InTest = true;
testCase.Mock.UseDefaults = false;


This comment was generated by todo based on a TODO comment in 247060a in #183. cc @cortex-lab.

@k1o0 k1o0 mentioned this pull request Jul 19, 2019
@k1o0
Copy link
Contributor Author

k1o0 commented Jul 19, 2019

For complicated reasons I'm merging despite the status check failing: due to compound commits the check run uses old tests before running the new ones.

@k1o0 k1o0 merged commit 3437b6c into dev Jul 19, 2019
k1o0 added a commit that referenced this pull request Jan 3, 2021
Fix for incorrect use of test attribute
k1o0 added a commit that referenced this pull request Jan 4, 2021
Fix for incorrect use of test attribute
@k1o0 k1o0 deleted the tests-update branch February 10, 2021 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants