Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client Side Testing #768

Closed
mihirsamdarshi opened this issue Jun 6, 2019 · 5 comments
Closed

Client Side Testing #768

mihirsamdarshi opened this issue Jun 6, 2019 · 5 comments

Comments

@mihirsamdarshi
Copy link
Collaborator

Attached are the 3 testing documents containing 20 tests each (all different) and one document containing 100 tests.

20 Test Files:
Test Document 1
Test Document 2
Test Document 3

100 Test File
Big Test Document

@kdahlquist
Copy link
Collaborator

I'm going to work through as many of the tests as I can before I leave to go home today.

@kdahlquist
Copy link
Collaborator

I haven't found any bugs yet, but one thing I'm noticing is that we may need to update the document for which conditions trigger a graph reload. Some things are not triggering a reload anymore (as we want).

For example, clicking the reset to the default normalization factor button no longer reloads the graph (as it should).

Another thing I'm noticing is that we should probably update our function availability table (as seen on wiki) with a longer list of functions. Right now the table is just listing what types of graphs are loaded, but for example, the testing document is telling me to reset the force graph parameter sliders when they haven't been changed. Essentially, we would be documenting the allowed states and actions.

@kdahlquist
Copy link
Collaborator

I finished going through the document: GRNsight.Test.1_20.pdf. The way these tests were organized, it was having me click the Reset Factor button, even though it didn't instruct me to type anything in the normalization factor field. The locking/unlocking force graph parameters check box works, but as noted in a different issue, the reset force graph parameters button is not working. Moving on to the next document.

@kdahlquist
Copy link
Collaborator

I went through the document: GRNsight.Test.2_24.pdf. I didn't quite do all the tests, but it seems like everything was behaving OK, except for the bug noted #777.

@kdahlquist kdahlquist removed their assignment Oct 7, 2019
@kdahlquist
Copy link
Collaborator

Closing this one because #575 is what we need to do now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants