Skip to content
This repository has been archived by the owner on Mar 1, 2023. It is now read-only.

Add "thenCheck()" to CompiledRepository. #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maxtroy
Copy link
Contributor

@maxtroy maxtroy commented Nov 21, 2016

"thenCheck()" can be used after another 'then' directive, which normally
generates the new value of the repository. Using a "thenCheck()" directive,
the would-have-been new value becomes a mere "candidate new value", and
can be vetoed by the check -- the clause that follows "thenCheck()" can
skip the update, end with another value, or even have the flow continue
onwards.

The test cases use two demo-style repositories that perform a typical task
of requesting a keyed item from the in-memory cache, then disk cache, then
network, showcasing the real-life use of this new directive.

"thenCheck()" can be used after another 'then' directive, which normally
generates the new value of the repository. Using a "thenCheck()" directive,
the would-have-been new value becomes a mere "candidate new value", and
can be vetoed by the check -- the clause that follows "thenCheck()" can
skip the update, end with another value, or even have the flow continue
onwards.

The test cases use two demo-style repositories that perform a typical task
of requesting a keyed item from the in-memory cache, then disk cache, then
network, showcasing the real-life use of this new directive.
@maxtroy maxtroy assigned ghost Nov 21, 2016
@ghost ghost assigned rosiebye Nov 21, 2016
@codecov-io
Copy link

codecov-io commented Nov 21, 2016

Current coverage is 97.38% (diff: 100%)

Merging #113 into master will increase coverage by 0.05%

@@             master       #113   diff @@
==========================================
  Files            39         39          
  Lines          1611       1645    +34   
  Methods           0          0          
  Messages          0          0          
  Branches        280        284     +4   
==========================================
+ Hits           1568       1602    +34   
  Misses           13         13          
  Partials         30         30          

Powered by Codecov. Last update d816f6d...50a5b4b

@ghost ghost requested a review from rosiebye December 20, 2016 17:47
@ghost ghost added the core label Dec 20, 2016
@ghost ghost added this to the v1.3 milestone Dec 20, 2016
@ghost ghost modified the milestones: v1.3, v1.4 Mar 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants