action #44039

/admin/test_suites is broken without login

Added by coolo about 1 year ago. Updated 9 months ago.

Status:ResolvedStart date:20/11/2018
Priority:NormalDue date:
Assignee:kraih% Done:

0%

Category:Concrete Bugs
Target version:Done
Difficulty:easy
Duration:

Description

The page doesn't show a 403 (I guess it should), but just barfs in javascript because:

Uncaught SyntaxError: Unexpected token ,

History

#1 Updated by coolo about 1 year ago

  • Project changed from openQA Tests to openQA Project
  • Category set to Concrete Bugs

#2 Updated by mkittler about 1 year ago

I guess it should

I remember doing a ticket which was about making admin pages publicly available in read-only mode. This would revert that effort. Likely fixing the JavaScript error is not a big deal so I'd say fix the error and keep the page publicly accessible.

#3 Updated by okurz about 1 year ago

+1

#4 Updated by kraih about 1 year ago

  • Status changed from New to In Progress

Fixing the JavaScript seems easy enough. There are three more uses of <%= is_admin %> that look really sketchy. Any pointers where a test for this could go?

#5 Updated by coolo about 1 year ago

I guess this is a good oportunity to get familiar with selenium - t/ui/13-admin.t logs in as first step. To test this, there needs to either an extra test file or more subtests before login

#6 Updated by kraih about 1 year ago

  • Status changed from In Progress to Resolved

#7 Updated by coolo 9 months ago

  • Target version changed from Ready to Done

Also available in: Atom PDF