Skip to content
Snippets Groups Projects

Issue #1149078: States API doesn't work with multiple select fields

Open Issue #1149078: States API doesn't work with multiple select fields
2 unresolved threads
2 unresolved threads

Merge request reports

Members who can merge are allowed to add commits.
Approval is optional
Test summary results are being parsed
Ready to merge by members who can write to the target branch.
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
149 149 *
150 150 * @prop {function} RegExp
151 151 * @prop {function} Function
152 152 * @prop {function} Number
  • 159 159 // The "reference" variable is a comparison function.
    160 160 return reference(value);
    161 161 },
    162 Array(reference, value) {
    163 // Make sure value is an array.
    164 if (!Array.isArray(value)) {
    165 return false;
    166 }
    167
    168 // The arrays values should match.
    169 return JSON.stringify(reference) === JSON.stringify(value);
  • In relation to the skipped tests, perhaps the style is significant in the underlying isVisible implementation for chromedriver?

    Perhaps we could assert the attribute was empty, then use something like executeScript to remove the attribute and see if that makes a difference.

    I agree though that it would be better to get this fixed and defer those tests to a follow up.

  • added 1 commit

    • 991fe1a6 - Issue #1149078: States API doesn't work with multiple select fields - Fix...

    Compare with previous version

  • added 1 commit

    • 4a587d80 - Issue #1149078: States API doesn't work with multiple select fields - Comments updates.

    Compare with previous version

  • Derek Wright added 1 commit

    added 1 commit

    Compare with previous version

  • Tavi Toporjinschi added 262 commits

    added 262 commits

    Compare with previous version

  • Please register or sign in to reply
    Loading