Commit graph

3430 commits

Author SHA1 Message Date
fb2300a799 Add Makefile for testing imag-store 2016-09-19 22:12:14 +02:00
72a95ee5da Rewrite config validity checker to return Result<()>
And add more detailed error kinds for config errors
2016-09-19 21:52:28 +02:00
83ebe88022 Add PreCreate hook test 2016-09-19 21:51:14 +02:00
1ac54ad29a Merge pull request #752 from matthiasbeyer/libimagstorestdhook/git-integration
libimagstorestdhook/git: integration
2016-09-19 14:39:15 +02:00
321b0493fb Merge pull request #749 from matthiasbeyer/libimagstorestdhook/git-reenable-warning-deny
libimagstorestdhook/git: reenable warning deny
2016-09-19 14:39:00 +02:00
4155924f85 Add test for Store::retrieve_for_module() 2016-09-19 11:25:36 +02:00
fd41fe5998 Fix Store::delete() for nonexistent IDs
If the ID does not exist, we should return an error instead of doing
nothing, shouldn't we?
2016-09-19 11:07:38 +02:00
8345ff8248 Add test to delete non-existent entries 2016-09-19 11:03:38 +02:00
52b2a4589d Add test to get non-existent entries 2016-09-19 11:01:56 +02:00
de6b641201 Merge branch 'master' into libimagstorestdhook/git-integration
This is a integration-merge, because the libimagstorestdhook/git branch
got really long and rebasing is not an option, as it would mainly
destroy history.

Using an integration branch is less hassle, really.
2016-09-19 10:44:57 +02:00
85097554e9 Add hook implementation for succeeding hook tests 2016-09-19 10:38:46 +02:00
0cdeeb4c55 Fix test backend implementation
We must return an error if the file is not found here.

This is because if we unwrap() here, we panic if the store action was
Store::retrieve() and there wasn't a file there. We then unwrap() on
None and panic because of this, causing all other tests to panic as
well because the Mutex gets corrupted.

The store handles FileNotFound errors on its own, so it is safe to
return the error here.
2016-09-19 10:37:28 +02:00
b25aa29fee Fix make call in travis config 2016-09-19 10:08:13 +02:00
Julian Ganz
d041d62d09 Replace invocation of "find" with test for a Makefile 2016-09-19 10:07:18 +02:00
Julian Ganz
c5d660d3f7 Make "test" a postfix to library targets rather than a prefix 2016-09-19 10:07:18 +02:00
Julian Ganz
f6dc039554 Make the binary tests depend on the binaries 2016-09-19 10:07:18 +02:00
Julian Ganz
0ff53a6925 Add general testing target 2016-09-19 10:07:18 +02:00
Julian Ganz
f9b4818607 Add a convenience target for binary tests 2016-09-19 10:07:18 +02:00
57114e159f Execute tests by calling another Makefile 2016-09-19 10:07:18 +02:00
12b6da5c90 Add first idea how to exec all the tests in the Makefile 2016-09-19 10:07:18 +02:00
f5d73ace0e Fix debug output, use action to generate msg 2016-09-19 10:02:40 +02:00
43bdedea8e Fix check whether we should checkout the ensure_branch
We should test this not at the top of the function, as if we do not have
to check out the ensure_branch, we still have to verify that there
actually is a branch checked out we have to ensure.

What I'm saying is: We might have to be on "master", but we might not
have to check it out if we aren't.

Therefor, this commit moves the location where we check whether we
should checkout the branch.
2016-09-19 10:02:40 +02:00
a74b46deed Add error kind for wrong branch checked out 2016-09-19 09:52:21 +02:00
352f468a27 Fix: We have to specify the full ref name in the cfg 2016-09-19 09:47:37 +02:00
76f92da898 Remove unused import 2016-09-19 09:28:22 +02:00
cf77c51afb Variable hasnt to be mutable 2016-09-19 09:28:22 +02:00
cae605d21b Runtime::ensure_cfg_branch_is_checked_out() should check whether config enforces this 2016-09-19 09:28:22 +02:00
73c0f080ac Change abort_on_repo_init_err() to take Value instead of Option<Value> 2016-09-19 09:28:22 +02:00
4a0def0ede Fix debug messages in Runtime::ensure_cfg_branch_is_checked_out() 2016-09-19 09:28:22 +02:00
a4cdb2b873 UpdateHook: Ensure the right branch is checked out 2016-09-19 09:28:22 +02:00
0f1c360981 DeleteHook: Ensure the right branch is checked out 2016-09-19 09:28:22 +02:00
81cc6c822b UpdateHook: Add config-based abort if repo init failed 2016-09-19 09:28:22 +02:00
a321ad941c DeleteHook: Add config-based abort if repo init failed 2016-09-19 09:28:22 +02:00
573745b9f3 Add error kind for repo initialization errors 2016-09-19 09:28:22 +02:00
27eaf4cf6e Revert "Disable lints"
This reverts commit 010109a3c3.
2016-09-19 09:28:22 +02:00
172614004a Merge pull request #745 from matthiasbeyer/libimagstorestdhook/git-commit-template
libimagstorestdhook/git: Provide better commit template message, including action and store id
2016-09-19 09:27:06 +02:00
68e1f4f61c Provide better commit template message, including action and store id 2016-09-18 19:23:09 +02:00
8558e6b241 Merge pull request #748 from matthiasbeyer/libimagstorestdhook/git-debug-output
libimagstorestdhook/git: More debug output
2016-09-18 18:55:19 +02:00
5384be1d99 Merge pull request #747 from matthiasbeyer/libimagstorestdhook/git-fix-warn-messages
libimagstorestdhook/git: fix warn messages
2016-09-18 18:08:58 +02:00
614b90e643 Merge pull request #746 from matthiasbeyer/libimagstorestdhook/git-refactor-error-mapping
libimagstorestdhook/git: refactor error mapping
2016-09-18 18:08:52 +02:00
df95338165 Merge pull request #744 from matthiasbeyer/libimagstorestdhook/git-remove-unused-error-kinds
libimagstorestdhook/git: Remove unused error kinds
2016-09-18 17:19:45 +02:00
556c464bbf Remove unused error kinds 2016-09-18 15:08:19 +02:00
bdbf136fef Add debug output for DeleteHook 2016-09-18 15:03:07 +02:00
c5f95e3538 Add debug output for UpdateHook 2016-09-18 15:00:36 +02:00
15dcb26fd2 Minify Runtime::new() by refactoring error tracing 2016-09-18 14:54:38 +02:00
07ff755ed7 Minify error handling in Store::_update with try!() 2016-09-18 14:52:55 +02:00
b288e938f9 Minify error handling in Store::update() 2016-09-18 14:52:00 +02:00
cb3a536adb Fix warn!() output to be action sensitive 2016-09-18 14:50:55 +02:00
9a4cdad9a1 Add StoreAction::as_commit_message() utility fn 2016-09-18 14:50:41 +02:00
40dbdaa5e1 Fix git hook registration fail warn message. 2016-09-18 14:44:08 +02:00