search

mhjort / trombi

Load testing library for testing stateful apps with Clojure
Eclipse Public License 1.0
235 stars 19 forks source link

Probable mistake in readme.md at `(assoc :user-name context)` #33

Closed yashaka closed 8 years ago

yashaka commented 8 years ago

Maybe I am wrong, but the following seems weird to me:

;step 1
(defn login [context]
  (go
    (let [user-name (login-to-system)]
      [true (assoc :user-name context)])))

shouldn't it be:

;step 1
(defn login [context]
  (go
    (let [user-name (login-to-system)]
      [true (assoc context :user-name user-name)])))

?

mhjort commented 8 years ago

You're right. Thanks for pointing out. I fixed the README.