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

There is no way to clean out old test run database tables #3868

Open
jenlampton opened this issue Jun 15, 2019 · 5 comments
Open

There is no way to clean out old test run database tables #3868

jenlampton opened this issue Jun 15, 2019 · 5 comments

Comments

@jenlampton
Copy link
Member

jenlampton commented Jun 15, 2019

Description of the bug
After a test fails half-way through, you get directed to the "error page". On the "error page" you will see a message that says;

Use the Clean environment button to clean-up temporary files and tables. 

But there is no "Clean environment button" provided on the error page.
Screen Shot 2019-06-14 at 10 07 47 PM

Steps To Reproduce
To reproduce the behavior:

  1. Run a test that fatals out half way through
  2. Click on 'error page'
  3. Notice the orange warning
  4. Look for the mysterious button
@herbdool
Copy link

The button should be way at the bottom. Isn't it?

@jenlampton
Copy link
Member Author

jenlampton commented Jun 25, 2019 via email

@klonos
Copy link
Member

klonos commented Jun 25, 2019

So the problem seems to be that this button is located at the bottom of the "List tests" page (admin/config/development/testing) - not in the "Test results" page.

We have 3 options:

  1. link to that page from the warning, and scroll to the bottom, where the "Clean test environment" fieldset is located
  2. duplicate the "Clean test environment" fieldset in the test results page.
  3. move the "Clean test environment" fieldset to the test results page.

Which one do we want?

@klonos
Copy link
Member

klonos commented Jun 25, 2019

...another point is that we have a "Clear results after each complete test suite run" checkbox in admin/config/development/testing/settings, which is on by default OOTB:

By default SimpleTest will clear the results after they have been viewed on the results page, but in some cases it may be useful to leave the results in the database. The results can then be viewed at admin/config/development/testing/[test_id]. The test ID can be found in the database, simpletest table, or kept track of when viewing the results the first time. Additionally, some modules may provide more analysis or features that require this setting to be disabled.

Which if I am reading things correctly, means that the "Clean test environment" fieldset should not be available when that setting is on. Right?

@jenlampton
Copy link
Member Author

So the problem seems to be that this button is located at the bottom of the "List tests" page (admin/config/development/testing) - not in the "Test results" page.

Sometimes it does on the test results page (your option 2), but it wasn't in this instance. Maybe we could solve the problem by always adding a link into the message (your option 1) Then we could remove it from this page always (for consistency) and take the mystery out of it.

Or, we could do option 3.

the "Clean test environment" fieldset should not be available when that setting is on. Right?

Well, in my case the tests failed half way through. I needed the button regardless of the setting.

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

3 participants