search

just3obad / smartsoft-12

Automatically exported from code.google.com/p/smartsoft-12
0 stars 0 forks source link

3.11 As an admin, I can reset a user account's password. #70

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Success scenarios:
The admin will send an email to the user with the new password (let’s say a 
string of random 10 characters) then the user will check this new password and 
he will be able to login using this password and he can change it whenever he 
wants.
Failure scenarios:

Original issue reported on code.google.com by gosgosakila on 10 May 2012 at 2:57

GoogleCodeExporter commented 9 years ago 
Why is this story New still?

Haven't the tests for them started yet!?

Original comment by guc@sakr.me on 13 May 2012 at 1:37

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1177.

Unit test done for this issue

Original comment by gosgosakila on 13 May 2012 at 3:02

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1184.

Added the functional test for this issue

Original comment by gosgosakila on 13 May 2012 at 5:18

GoogleCodeExporter commented 9 years ago 
A kind reminder that our deadline is in less than 2 hours.

Original comment by guc@sakr.me on 13 May 2012 at 8:19

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1227.

Adding RED to the tests name

Original comment by gosgosakila on 13 May 2012 at 9:55

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1508.

Added the email view

Original comment by gosgosakila on 15 May 2012 at 9:55

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1512.

Added the method force_reset_password in the emailer and its documentation

Original comment by gosgosakila on 15 May 2012 at 10:03

GoogleCodeExporter commented 9 years ago 
see r1485

Original comment by christin...@gmail.com on 16 May 2012 at 8:43

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1582.

Called the force_reset_pasword method in users_controller and added the flash 
success bar

Original comment by gosgosakila on 16 May 2012 at 10:16

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1588.

Modified the method force_reset_password in the users_controller

Original comment by gosgosakila on 16 May 2012 at 11:57

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1590.

Modified the show page to render the success flash

Original comment by gosgosakila on 17 May 2012 at 12:02

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1591.

Added the view of the mail sent to the user

Original comment by gosgosakila on 17 May 2012 at 12:06

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1592.

Added the view of the mail sent to the user

Original comment by gosgosakila on 17 May 2012 at 12:06

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1599.

Labels Doc-Ready
Added the Documentation for the send_forced_password method

Original comment by gosgosakila on 17 May 2012 at 12:17

GoogleCodeExporter commented 9 years ago 
Sorry about forgetting : after labels in the last commit.

Original comment by gosgosakila on 17 May 2012 at 12:19

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1749.

Edited the documentation

Original comment by gosgosakila on 18 May 2012 at 4:29

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1823.

Added the UML for this issue

Original comment by gosgosakila on 18 May 2012 at 9:26

GoogleCodeExporter commented 9 years ago

Original comment by mouaz.alabsawi@gmail.com on 19 May 2012 at 12:55

GoogleCodeExporter commented 9 years ago 
so far , scenario is working well , it sends an email , but tests needs to be 
greenified to make sure scenario goes well .

Original comment by jailan.s...@gmail.com on 19 May 2012 at 3:30

GoogleCodeExporter commented 9 years ago 
Routing error in the test.
Good job testing the mailer I just need the error to be fixed in order to make 
sure of the validity of the test.

Original comment by elmen...@gmail.com on 19 May 2012 at 8:02

GoogleCodeExporter commented 9 years ago 
Identation in the controller the whole method

Original comment by mina.wor...@gmail.com on 20 May 2012 at 12:55

GoogleCodeExporter commented 9 years ago 
and why are you calling:
@logs = @user.get_recent_activity(Time.zone.now)
@friends = @user.friends
 @interests = @user.added_interests

Original comment by mina.wor...@gmail.com on 20 May 2012 at 1:23

GoogleCodeExporter commented 9 years ago 
Documentation should be in the format:
= begin
Method description
Parameters (inputs, outputs)
Author: Gasser
= end
In addition to simple in line documentation to inside the methods

Original comment by bassemaw...@gmail.com on 20 May 2012 at 7:13

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1924.

Fixed the tests of this issue

Original comment by gosgosakila on 20 May 2012 at 10:33

GoogleCodeExporter commented 9 years ago 
@mina.worksace ==> I put those variables because I redirect to the user profile 
page again so I should initialize those instance variables my self as they are 
used by other methods. The indentation is DONE.

Original comment by gosgosakila on 20 May 2012 at 11:56

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1947.

Fixed the reviewers comments

Original comment by gosgosakila on 20 May 2012 at 11:57

GoogleCodeExporter commented 9 years ago 
in the users controller tests
the first test passes, however the second gives the following error:

No route matches "/force_reset_password/1"

in "admin should reset users password RED"

Original comment by elmen...@gmail.com on 23 May 2012 at 1:37

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r2243.

Deleted the old test with the error

Original comment by gosgosakila on 23 May 2012 at 2:11

GoogleCodeExporter commented 9 years ago 
Good job ;)

Original comment by elmen...@gmail.com on 23 May 2012 at 2:35

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r2320.

Edited the mail sent to the user

Original comment by gosgosakila on 23 May 2012 at 6:46

GoogleCodeExporter commented 9 years ago

Original comment by bassemaw...@gmail.com on 23 May 2012 at 6:53

GoogleCodeExporter commented 9 years ago

Original comment by mina.wor...@gmail.com on 23 May 2012 at 7:45

GoogleCodeExporter commented 9 years ago 
I get this whenever I try to reset a user's Password

Original comment by jailan.s...@gmail.com on 23 May 2012 at 9:24

Attachments:

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r2402.

Added green to the tests

Original comment by gosgosakila on 23 May 2012 at 9:52

GoogleCodeExporter commented 9 years ago 
my bad , didn't add migrations

Original comment by jailan.s...@gmail.com on 23 May 2012 at 9:53

GoogleCodeExporter commented 9 years ago

Original comment by yahiaelg...@gmail.com on 23 May 2012 at 11:10