Skip to content
Browse files

Update the round trip learner data expectations. Document that.

  • Loading branch information...
1 parent 8dbeab5 commit bacb3979ff774d4bbf91eaffefc5477e56a2bd66 @knowuh knowuh committed Apr 5, 2012
Showing with 4 additions and 3 deletions.
  1. +3 −2 README.md
  2. +1 −1 apps/my_system/tests/jasmine/integration/round_trip_learner.js
View
5 README.md
@@ -162,8 +162,9 @@ If you create or modify persistent properties, you will also want to create a mi
0. Increment the MySytem.learnerData version found `./apps/my_system/core.js`
1. Create the appropriate migration file in `./apps/my_system/migrations/migrate_learner_data_*.js`, this file should document the model changes.
2. Ensure that the migration tests in `tests/jasmine/migrations` runs without error:
- * http://localhost:4020/sproutcore/tests#my_system&test=jasmine/migrations/migrate_learner_data_behavior
- *
+ * http://localhost:4020/sproutcore/tests#my_system&test=jasmine/migrations/migrate_learner_data_behavior.js
+ * http://localhost:4020/sproutcore/tests#my_system&test=jasmine/migrations/migrate_learner_data_test_cases.js
+3. update the test here: ./apps/my_system/tests/jasmine/integration/round_trip_learner.js
### Building as WISE4 step ###
View
2 apps/my_system/tests/jasmine/integration/round_trip_learner.js
@@ -120,6 +120,6 @@ describe("The learner data", function(){
var parsedLearnerData = JSON.parse(savedLearnerData);
// This should be updated if you change the learner data version
- expect(parsedLearnerData.version).toBe(4);
+ expect(parsedLearnerData.version).toBe(5);
});
});

0 comments on commit bacb397

Please sign in to comment.
Something went wrong with that request. Please try again.