genenetwork / genenetwork2

GeneNetwork (2nd generation)
http://gn2.genenetwork.org/
GNU Affero General Public License v3.0
34 stars 24 forks source link

Duplicating the mechanize tests in python #290

Closed fredmanglis closed 6 years ago

fredmanglis commented 6 years ago

The mechanize tests can be found at the following urls:

The entry point for the new tests is test/requests/test-website.py in this fork of the repository

fredmanglis commented 6 years ago

Running the test for R/qtl mapping causes the main system to throw the exception indicated in output snippet in the following file:

report_on_r_qtl_mapping_tests.txt

I still don't understand what the mapping does, so I'll skip this for now, and come back to it after implementing the other tests.

pjotrp commented 6 years ago

@fredmanglis if you tell me how to duplicate the error I can take a look.

Be good if we can duplicate the exact error on the production site http://gn2.genenetwork.com/. But Africa probably does not have access... Otherwise, see if you get the same error through a web link on your local machine. Thanks.

fredmanglis commented 6 years ago

@pjotrp

Get my latest changes, then run something like:

env GN2_PROFILE=~/guix_profiles/gn2 ./bin/genenetwork2 \
../custom_settings.py -c \
../test/requests/test-website.py http://localhost:5003 -m

You'll note that the tests fail at the R/qtl mapping tool selection stage.

Now if you look at the output of the flask application (serving at url given), you should see a similar error to the one in the given file.

fredmanglis commented 6 years ago

@pjotrp I was able to reproduce the error, on the browser:

Steps

zsloan commented 6 years ago

Hm, this seems to work okay on production and my branch.

On Mon, Mar 19, 2018 at 9:58 AM, Muriithi Frederick Muriuki < notifications@github.com> wrote:

@pjotrp https://github.com/pjotrp I was able to reproduce the error, on the browser: Steps

  • Go to http://localhost:5003 (or whichever url you'll be using)
  • Leave "Get Any" blank
  • Enter MEAN=(15 16) LRS=(23 46) in "Combined" field
  • Click "Search"
  • Click on first link (1443823_s_at) and wait for page to complete loading
  • Expand "Mapping Tools" accordion
  • Click on the "R/qt" link
  • Click the "Compute" button
  • A new tab pops up, and after a while, the exception is shown.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/genenetwork/genenetwork2/issues/290#issuecomment-374242156, or mute the thread https://github.com/notifications/unsubscribe-auth/ABsEmGXob5Dqw1jOLBQ774t842Z_Ww5xks5tf8ebgaJpZM4SvDb- .

pjotrp commented 6 years ago

Yeah, it is probably a data problem. We are running on the small database and old genotype files (which should match). I'll look at it soon.

fredmanglis commented 6 years ago

Navigation tests have been ignored for the time being, since they rely on Javascript to render the page. To run them would involve use of selenium and its ilk, which would require definition of packages for them on Guix.