[ckan-changes] commit/ckan: 2 new changesets

Mon Sep 5 09:15:31 UTC 2011

2 new changesets in ckan:

http://bitbucket.org/okfn/ckan/changeset/1b77ce8ce560/
changeset:   1b77ce8ce560
branch:      release-v1.4.3
user:        dread
date:        2011-09-05 10:41:28
summary:     [tests]: #1284 Adds tests for the actual fix, which was cset:1952445d2802.
affected #:  1 file (503 bytes)

--- a/ckan/tests/functional/test_user.py	Sun Sep 04 17:22:42 2011 +0100
+++ b/ckan/tests/functional/test_user.py	Mon Sep 05 09:41:28 2011 +0100
@@ -191,6 +191,7 @@
         fullname = u'Test Create'
         password = u'testpassword'
         assert not model.User.by_name(unicode(username))
+        rev_id_before_test = model.repo.youngest_revision().id
 
         offset = url_for(controller='user', action='register')
         res = self.app.get(offset, status=200)
@@ -222,6 +223,11 @@
         assert_equal(user.fullname, fullname)
         assert user.password
 
+        # no revision should be created - User is not revisioned
+        rev_id_after_test = model.repo.youngest_revision().id
+        assert_equal(rev_id_before_test, rev_id_after_test)
+
+
     def test_user_create_unicode(self):
         # create/register user
         username = u'testcreate4'
@@ -381,6 +387,7 @@
                                          password='letmein'))
             model.repo.commit_and_remove()
             user = model.User.by_name(unicode(username))
+        rev_id_before_test = model.repo.youngest_revision().id
 
         # edit
         new_about = u'Changed about'
@@ -416,6 +423,10 @@
         main_res = self.main_div(res)
         assert new_about in main_res, main_res
 
+        # no revision should be created - User is not revisioned
+        rev_id_after_test = model.repo.youngest_revision().id
+        assert_equal(rev_id_before_test, rev_id_after_test)
+
     def test_user_edit_no_password(self):
         # create user
         username = 'testedit2'


http://bitbucket.org/okfn/ckan/changeset/1f958df008b2/
changeset:   1f958df008b2
user:        dread
date:        2011-09-05 10:43:11
summary:     [merge] from release-1.4.3.
affected #:  3 files (503 bytes)

--- a/ckan/tests/functional/test_user.py	Sun Sep 04 17:22:42 2011 +0100
+++ b/ckan/tests/functional/test_user.py	Mon Sep 05 09:43:11 2011 +0100
@@ -191,6 +191,7 @@
         fullname = u'Test Create'
         password = u'testpassword'
         assert not model.User.by_name(unicode(username))
+        rev_id_before_test = model.repo.youngest_revision().id
 
         offset = url_for(controller='user', action='register')
         res = self.app.get(offset, status=200)
@@ -222,6 +223,11 @@
         assert_equal(user.fullname, fullname)
         assert user.password
 
+        # no revision should be created - User is not revisioned
+        rev_id_after_test = model.repo.youngest_revision().id
+        assert_equal(rev_id_before_test, rev_id_after_test)
+
+
     def test_user_create_unicode(self):
         # create/register user
         username = u'testcreate4'
@@ -381,6 +387,7 @@
                                          password='letmein'))
             model.repo.commit_and_remove()
             user = model.User.by_name(unicode(username))
+        rev_id_before_test = model.repo.youngest_revision().id
 
         # edit
         new_about = u'Changed about'
@@ -416,6 +423,10 @@
         main_res = self.main_div(res)
         assert new_about in main_res, main_res
 
+        # no revision should be created - User is not revisioned
+        rev_id_after_test = model.repo.youngest_revision().id
+        assert_equal(rev_id_before_test, rev_id_after_test)
+
     def test_user_edit_no_password(self):
         # create user
         username = 'testedit2'

Repository URL: https://bitbucket.org/okfn/ckan/

--

This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.


