feat: update sleep and container name to work on both Mac and Linux

[jcstein]

Overview

Replaces #5

Problem 1

Example on macOS when container is named nitro-testnode-da-1 :

✔ Container nitro-testnode-da-1              Started                   0.6s
Waiting for http://localhost:26659/header/1 to come up..........................................................................Done!
Error response from daemon: No such container: nitro-testnode_da_1

Example on Ubuntu when container is named nitro-testnode_da_1 :

✔ Container nitro-testnode_da_1              Started                   0.6s
Waiting for http://localhost:26659/header/1 to come up..........................................................................Done!
Error response from daemon: No such container: nitro-testnode-da-1

Solution 1

Give the container a name: da-celestia, keeping things standard across different OS.

 ✔ Container da-celestia                      Started                   0.1s
Waiting for http://localhost:26659/header/1 to come up...........................................................Done!
== Generating l1 keys

4763e34

Problem 2

Using sleep 100s on Mac does not run the same way as ubuntu, also, the docker file has 2 different syntax for 2 different sleep commands, with and without the s.

nitro-testnode/test-node.bash

Line 262 in d5635fb

sleep 0.25

nitro-testnode/test-node.bash

Line 343 in d5635fb

sleep 100s

Solution 2

Use standard sleep x from the original script, without the s, allowing this to run on all machines.

1d53c49

Checklist

• New and updated code has appropriate documentation
• New and updated code has new and/or updated testing
• Required CI checks are passing
• Visual proof for any user facing features like CLI or documentation updates
• Linked issues closed with keywords

[jcstein]

Tested successfully, see full logs

[jcstein]

Reminder: after this is merged, the commit of nitro-testnode on nitro will need updated

[Ferret-san]