mozilla / experimenter

A web application for managing user experiments for Mozilla Firefox.
https://experimenter.services.mozilla.com
Mozilla Public License 2.0
114 stars 175 forks source link

The "Server Error (500)" is displayed if the Branch - "Name" section has no value inserted in the input field when saved #1203

Closed Softvision-RemusDranca closed 5 years ago

Softvision-RemusDranca commented 5 years ago

[Affected versions]:

[Affected Platforms]:

[Prerequisites]:

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites and navigate to "https://stage.experimenter.nonprod.dataops.mozgcp.net/".
  2. Open the created experiment from prerequisites.
  3. Click the "Edit" button from "Branches" section.
  4. Delete the value from the "Name" input field.
  5. Click the "Save Draft" button and observe the behavior.

[Expected result]:

[Actual result]:

[Notes]:

jaredlockhart commented 5 years ago

Ah yes this is a regression caused by https://github.com/mozilla/experimenter/issues/1129

We should add a new test case to recreate this and then make sure that field is non-empty before we try to do the slug validation.

Softvision-RemusDranca commented 5 years ago

I have verified this issue and is no longer reproducible on Experimenter stage "https://stage.experimenter.nonprod.dataops.mozgcp.net/" website after the v2.28.1 stage deployment. However, the wrong error message is displayed after saving an experiment with no value inserted in the Branch - Name section. I opened a new issue #1253 reflecting this behavior. Tested on Windows 10x64, Mac 10.14 and Arch Linux 4.16.6 x64.