Skip to content

Commit

Permalink
Merge pull request #1209 from eclipse-pass/russ-715-opt-locking
Browse files Browse the repository at this point in the history
[pass-ui] Submission and Deposit optimistic locking
  • Loading branch information
rpoet-jh authored Sep 25, 2023
2 parents f25f4a8 + a61696f commit f38e022
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions app/models/submission.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ export default class SubmissionModel extends Model {
defaultValue: null,
})
submitterEmail;
@attr('number') version;

@belongsTo('user') submitter;
@belongsTo('publication') publication;
Expand Down
4 changes: 3 additions & 1 deletion tests/unit/services/submission-handler-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ module('Unit | Service | submission-handler', (hooks) => {

let submission = EmberObject.create({
id: '0',
version: 3,
submitter: {
id: 'submitter:test-id',
},
Expand All @@ -62,14 +63,15 @@ module('Unit | Service | submission-handler', (hooks) => {
let files = A();
let comment = 'blarg';

assert.expect(13);
assert.expect(14);

return service
.get('submit')
.perform(submission, publication, files, comment)
.then(() => {
assert.false(submission.get('submitted'));
assert.strictEqual(submission.get('submissionStatus'), 'approval-requested');
assert.strictEqual(submission.get('version'), 3);

// web-link repo should not be removed
assert.strictEqual(submission.get('repositories.length'), 2);
Expand Down

0 comments on commit f38e022

Please sign in to comment.