Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
3 changes: 1 addition & 2 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
*.DS_Store
.idea
rsapass*
25 changes: 17 additions & 8 deletions lib/desugaring.rb
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,9 @@ def all_the_sugar(recipients, event, message)
# Copy the contents of the previous method here and remove this sugar.
#
def desugared_poetry(recipients, event, message)
implement_me!
mail(message,
to: recipients.map(&:email),
subject: "You’re invited to #{event.title} on #{event.date}")
end

# Ruby allows you to pass arguments identified by name instead of just by position. They are really just
Expand All @@ -36,7 +38,8 @@ def desugared_poetry(recipients, event, message)
# Copy the contents of the previous method here and remove this sugar.
#
def desugared_named_args(recipients, event, message)
implement_me!
mail(message,{to: recipients.map(&:email),
subject: "You’re invited to #{event.title} on #{event.date}"})
end

# Ruby’s general syntax for hashes is `{key => value, key => value, ...}`. Because it is so common to use
Expand All @@ -51,7 +54,8 @@ def desugared_named_args(recipients, event, message)
# Copy the contents of the previous method here and remove this sugar.
#
def desugared_symbol_keys(recipients, event, message)
implement_me!
mail(message,{:to => recipients.map(&:email),
:subject => "You’re invited to #{event.title} on #{event.date}"})
end

# You may be wondering how `map(&:email)` works. When you precede the last argument of a method call with
Expand All @@ -70,7 +74,8 @@ def desugared_symbol_keys(recipients, event, message)
# Copy the contents of the previous method here and remove this sugar.
#
def desugared_attr_proc(recipients, event, message)
implement_me!
mail(message,{:to => recipients.map{ |person| person.email },
:subject => "You’re invited to #{event.title} on #{event.date}"})
end

# You may recall from the Ruby koans that when you put `#{something}` in a `"`-delimited string, Ruby will
Expand All @@ -88,7 +93,8 @@ def desugared_attr_proc(recipients, event, message)
# Copy the contents of the previous method here and remove this sugar.
#
def desugared_interpolation(recipients, event, message)
implement_me!
mail(message,{:to => recipients.map{ |person| person.email },
:subject => "You’re invited to " + event.title.to_s + ' on ' + event.date.to_s})
end

# Ruby tracks local variables lexically at compile time. Wherever you say `x = y`, the compiler assumes that
Expand All @@ -110,7 +116,8 @@ def desugared_interpolation(recipients, event, message)
# (Think: which names are local variables, and which are not?)
#
def desugared_implicit_self(recipients, event, message)
implement_me!
self.mail(message,{:to => recipients.map{ |person| person.email },
:subject => "You’re invited to " + event.title.to_s + ' on ' + event.date.to_s})
end

# In Ruby, unlike Python, there are no properties distinct from method calls. When you say `x.y`, you are
Expand All @@ -131,7 +138,8 @@ def desugared_implicit_self(recipients, event, message)
# but structurally quite similar!
#
def desugared_implicit_parens(recipients, event, message)
implement_me!
self.mail(message,{:to => recipients.map{ |person| person.email() },
:subject => "You’re invited to " + event.title.to_s() + ' on ' + event.date.to_s()})
end

# In Ruby, every value is an object and every action is a method call. That includes operators. A binary
Expand All @@ -153,7 +161,8 @@ def desugared_implicit_parens(recipients, event, message)
# get added before the things on the right. (a + b + c) means ((a + b) + c), NOT (a + (b + c)).
#
def desugared_operators(recipients, event, message)
implement_me!
self.mail(message,{:to => recipients.map{ |person| person.email() },
:subject => ((("You’re invited to ".+(event.title.to_s())).+(' on ')).+(event.date.to_s()))})
end

# Compare that to the version at the top.
Expand Down
4 changes: 4 additions & 0 deletions lib/retail_transaction.rb
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ def paid?
state :processing_payment
state :payment_declined
state :settled
state :refunded

event :check_out do
transitions from: :ringing_up, to: :collecting_payment,
Expand All @@ -61,5 +62,8 @@ def paid?
event :payment_declined do
transitions from: :processing_payment, to: :payment_declined
end
event :refund do
transitions from: :settled, to: :refunded
end
end
end
40 changes: 40 additions & 0 deletions test/retail_transaction_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,10 @@
tx.check_out!
end

it "cannot be refunded" do
assert_invalid_transition { tx.refund! }
end

it "cannot add more items" do
assert_raises do
tx.add_item("roller skates")
Expand All @@ -74,6 +78,10 @@
tx.process_payment!
end

it "cannot be refunded" do
assert_invalid_transition { tx.refund! }
end

it "cannot add more items" do
assert_raises do
tx.add_item("roller skates")
Expand Down Expand Up @@ -149,8 +157,40 @@
tx.payment_authorized!
end

it "cannot be reopened" do
assert_invalid_transition { tx.reopen! }
end
it "can be refunded" do
tx.refund!
assert_equal true, tx.refunded?
end
end

describe "that is refunded" do
before(:each) do
tx.add_item("bobcat")
tx.check_out!
tx.payment_info = "15 cents and a nail"
tx.process_payment!
tx.payment_authorized!
tx.refund!
end

it "cannot be refunded a second time" do
assert_invalid_transition { tx.refund! }
end

it "cannot be reopened" do
assert_invalid_transition { tx.reopen! }
end
end
end