Fix Testing docs

[amitaibu]

No description provided.

[amitaibu]

Compiler complained fetch doesn't exist in this package.

[mpscholten]

The fetch functions have moved from IHP.QueryBuilder to IHP.Fetch

[amitaibu]

Changed to make it easier to copy/ paste

[amitaibu]

I'm couldn't install hspec locally (Ubuntu 20), but also couldn't make it work on GitPod.io .

I've shared a workspace. Maybe others could have a look and help find the problem, without needing to install anything locally. (GitPod spinned of https://github.com/gitpod-io/template-ihp)

[mpscholten]

It got this error when trying to access the gitpod:

What command are you using to run the test?

[amitaibu]

What command are you using to run the test?

hspec

[amitaibu]

It got this error when trying to access the gitpod:

Here are the steps how I've tried to add hspec

1. Spin new GitPod out of https://github.com/gitpod-io/template-ihp
2. Add hspec to default.nix
3. Run make -B .envrc
4. Try to execute hspec - doesn't recognize command

[amitaibu]

Here's a Workspace snapshot, maybe that would work - https://gitpod.io#snapshot/76b48fa0-9e16-49af-8370-1235db2bde39

[mpscholten]

Thanks :)

Try to execute hspec - doesn't recognize command

I'm not aware that hspec is callable from the CLI. In our projects we typically have a Test/Main.hs file that imports and runs all the tests. Similar like this one in IHP https://github.com/digitallyinduced/ihp/blob/master/Test/Main.hs

Then I run ghci and :l Test/Main and main to run the Test/Main.hs module.

[amitaibu]

Alright, that makes sense. So what commands should we mention in https://ihp.digitallyinduced.com/Guide/testing.html#running-a-test (Running a Test)

[amitaibu]

I mean:

nix-shell
ghci
:l Test/Main
main

And remove the hspec spec?

[amitaibu]

:l Test/Main is actually for IHP lib. Not for a project

[mpscholten]

Yes 👍

:l Test/Main is actually for IHP lib. Not for a project

Should be the same for the actual project itself. Or then we should update the testing docs to make clear that tests should be stored in the Test/ directory of the project (which needs to be created on first use)

[amitaibu]

Yeah, PR is indeed trying to fix the docs 😸 . So should I copy the Test/Main.hs from IHP lib?

[mpscholten]

So should I copy the Test/Main.hs from IHP lib?

Yes that sounds good to me 👍

[amitaibu]

Getting closer.. I'm trying to add it here as a playground, but now have

Test/Controller/BidSpec.hs:20:19: error:
    Variable not in scope: makeConfig :: IO ConfigBuilder
   |
20 | spec = beforeAll (makeConfig >>= mockContext WebApplication) do

However I couldn't fund makeConfig - where is it coming from?

[mpscholten]

However I couldn't fund makeConfig - where is it coming from?

Should be the same as in Config.hs, so something like this:

makeConfig :: ConfigBuilder
makeConfig = do
    option Development
    option (AppHostname "localhost")

[amitaibu]

Note to self, add comment about

ghci
GHCi, version 8.10.3: https://www.haskell.org/ghc/  :? for help
*** WARNING: . is writable by someone else, IGNORING!
Suggested fix: execute 'chmod go-w .'

That should become

GHCi, version 8.10.3: https://www.haskell.org/ghc/  :? for help
package flags have changed, resetting and loading new packages...
Loaded GHCi configuration from /home/amitaibu/Sites/Haskell/ihp/blog/.ghci

[amitaibu]

PR is actually outdated with your changes - I was just working on an updated one :)

[mpscholten]

Ok, feel free to make a new one 👍