Skip to content

ExpectThunk proposal #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 24 commits into from
Jun 13, 2019
+241 −2
Merged

ExpectThunk proposal #19

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
04b9cff
ExpectThunk proposal
obj-p Feb 14, 2019
dc21ace
wowza trailing spaces
obj-p Feb 14, 2019
6d06241
adding ability to pass custom assertion block
obj-p Feb 14, 2019
0118c5e
refactoring ExpectThunk away from subclassing, and various tidbits
obj-p Feb 14, 2019
0bf5468
using createThunkMiddleware in ExpectThunk
obj-p Feb 19, 2019
fc20af6
renaming ExpectThunk members
obj-p Feb 19, 2019
aa3e1ce
adding boundary guard and some cleanup
obj-p Feb 20, 2019
a674e51
code style changes
obj-p Feb 20, 2019
6b86566
trailing whitespace
obj-p Feb 20, 2019
0b543e4
removing type as it's inferred
obj-p Feb 20, 2019
5e154b5
adding podspec
obj-p Feb 20, 2019
7e638f1
adding public accessors to getsState and run
obj-p Feb 20, 2019
754dbce
updating podspec
obj-p Feb 20, 2019
063b055
removing podspec and adding subspec
obj-p Feb 21, 2019
c313633
adding preprocessor flag
obj-p Feb 21, 2019
4d8c279
using isEmpty for fun
obj-p Feb 21, 2019
2440ec2
adding a blurb on expect-thunk in README.md
obj-p Feb 25, 2019
a671f34
removing equatable from FakeAction
obj-p Apr 17, 2019
0922abe
New wait/run API as well as dispatched record
obj-p May 18, 2019
8618792
adding wait failed
obj-p May 18, 2019
30f9a96
updating failure descriptions
obj-p May 18, 2019
ff7234f
adding blurb about ExpectThunk subspec
obj-p Jun 11, 2019
5bf1d69
Adding CHANGELOG comment for ExpectThunk
obj-p Jun 13, 2019
f272dc9
add PR reference
DivineDominion Jun 13, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
using isEmpty for fun
  • Loading branch information
@obj-p
obj-p committed Feb 21, 2019
commit 4d8c279f07d22a5bcde3bd884041687cc9fd4f33
6 changes: 3 additions & 3 deletions ReSwift-ThunkTests/ExpectThunk.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ public class ExpectThunk<State: StateType> {
private var dispatchAssertions = [DispatchAssertion]()
private var dispatch: DispatchFunction {
return { action in
guard self.dispatchAssertions.count > 0 else {
guard self.dispatchAssertions.isEmpty == false else {
return
}
self.dispatchAssertions.remove(at: 0)(action)
if self.dispatchAssertions.count == 0 {
if self.dispatchAssertions.isEmpty {
self.expectation.fulfill()
}
}
Expand All @@ -31,7 +31,7 @@ public class ExpectThunk<State: StateType> {
private var providedStates = [State]()
private var getState: () -> State? {
return {
return self.providedStates.count > 0 ? self.providedStates.removeFirst() : nil
return self.providedStates.isEmpty ? nil : self.providedStates.removeFirst()
}
}
private let thunk: Thunk<State>
Expand Down