Closed
Bug 996248
Opened 11 years ago
Closed 7 years ago
[FxAccounts] Move refreshAuthentication test from IAC-API TestFXA app to RP-API UITest app
Categories
(Firefox OS Graveyard :: FxA, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: njpark, Unassigned)
Details
(Whiteboard: [fxa4fxos2.0])
No description provided.
Reporter | ||
Comment 1•11 years ago
|
||
STR:
- Log in to Firefox Accounts, and the use the FxA Test app and click 'Refresh Authentication UI'
- Enter the correct password.
- (Alternatively) enter the wrong password:
Expected:
- The dialog that shows the correct workflow should be displayed.
Actual:
- 'The user is already signed in' message shows.
Version Info:
│ Gaia f3abbd2d0a60f1a1618db93f8b1957cae6de379c │
│ Gecko https://hg.mozilla.org/mozilla-central/rev/215080b813a7 │
│ BuildID 20140414040203 │
│ Version 31.0a1
Updated•11 years ago
|
Comment 2•11 years ago
|
||
I believe that we have no plans to exercise refreshAuthentication through the IAC API in the 2.0 timeframe. refreshAuth will be used by FMD and possible Marketplace via navigator.mozId, and will work* once bug 994934 lands (i.e. sometime today). Logout, which does use the IAC API, does not require forceAuth** (see bug 949100 ).
I therefore propose that we close this issue by removing the refreshAuthentication functionality from the TestFxA app. Jared and Jed?
* except that right now a failed challenge DOES log you out, which contradicts the user stories. I'll file a follow-up bug.
** forceAuth is just refreshAuth with a grace period of 0.
Flags: needinfo?(jparsons)
Flags: needinfo?(6a68)
Comment 3•11 years ago
|
||
(In reply to Sam Penrose from comment #2)
> I therefore propose that we close this issue by removing the
> refreshAuthentication functionality from the TestFxA app. Jared and Jed?
Sure, maybe we can file under 2.0 gaia nice-to-haves, since it's not hurting anything, and we haven't cleared the 2.0 blockers yet?
Flags: needinfo?(6a68)
Comment 4•11 years ago
|
||
I followed Jared's suggestion re parent bug. I would still like to delete the code from (or at least hide the UI in) the Test FxA app, so that the latter is a faithful representation of what we are trying to do.
Comment 5•11 years ago
|
||
That sounds good to me. Remove the test and replace it with a comment pointing to the new bug?
Flags: needinfo?(jparsons)
Updated•11 years ago
|
Summary: [FxAccounts] 'user is already signed in' message shows when undergoing forced authentication → [FxAccounts] Move refreshAuthentication test from IAC-API TestFXA app to RP-API UITest app
Comment 6•11 years ago
|
||
Experimenting with a project whiteboard tag, "[fxa4fxos2.0]". Feedback welcome.
Whiteboard: [fxa4fxos2.0]
Updated•10 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Assignee: spenrose → nobody
Comment 7•7 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•