TESTING: Demo/Large Explore Tests

gjwgit / rattleng

The New Generation R Analytics Desktop App

GNU General Public License v3.0

20 stars 8 forks source link

TESTING: Demo/Large Explore Tests #343

Open gjwgit opened 2 months ago

junhaow1 commented 1 month ago

https://github.com/gjwgit/rattleng/tree/kev/343_test Hi @gjwgit

[x] demo test added
[ ] The large test is not implemented, and I'm not sure which 'Input' and 'Second' parameters to choose. I tried several options, but it still failed. Maybe we need to fix this feature? I don't think this should happen in the first place.

junhaow1 commented 1 month ago

https://github.com/gjwgit/rattleng/tree/kev/343_test working branch

junhaow1 commented 1 month ago

https://github.com/gjwgit/rattleng/tree/kev/343_test Hi @gjwgit

[x] demo test added

[ ] The large test is not implemented, and I'm not sure which 'Input' and 'Second' parameters to choose. I tried several options, but it still failed. Maybe we need to fix this feature? I don't think this should happen in the first place.

in progress now

gjwgit commented 1 month ago

I'll have a look but I think the tests all need numeric variables - the GUI seems to be allowing all variables. That needs to be fixed as part of this test task.

junhaow1 commented 1 month ago

I'll have a look but I think the tests all need numeric variables - the GUI seems to be allowing all variables. That needs to be fixed as part of this test task.

Yeah numeric variables are working now. I am writing the test.

gjwgit commented 1 month ago

Thanks. Please update the tests/config.dart to limit the choices to numeric variables and also include in your test a test for any non-numeric variables in the list of available variables. Thanks.

junhaow1 commented 1 month ago

Thanks. Please update the tests/config.dart to limit the choices to numeric variables and also include in your test a test for any non-numeric variables in the list of available variables. Thanks.

sounds good , on it now

junhaow1 commented 1 month ago

Thanks. Please update the tests/config.dart to limit the choices to numeric variables and also include in your test a test for any non-numeric variables in the list of available variables. Thanks.

https://github.com/gjwgit/rattleng/issues/382 created , will fix this in another branch. Still looking for ways to filter the options.

junhaow1 commented 1 month ago

Hi @gjwgit , the working branch is https://github.com/gjwgit/rattleng/tree/kev/343_test

the blocker is I could not select "height" as the first dropdown and "weight" as the second dropdown at the same time. I tried few ways and still failed.

related code: integration_test/explore_tests_large_test.dart line 57-86