Skip to content

Commit

Permalink
Test activate shortcut in the tutorial test
Browse files Browse the repository at this point in the history
Currently the implementation of the activate shortcut (`-a`) for `workspace
create` is fragile: it relies on the stdout from the create command, so
any redundant log entries would break it. There's no current end-to-end
test to catch its potential breakage (challenging to test given the
impl lives in the shell wrapper layer.) So piggy-back on this tutorial
e2e test to ensure its working.
  • Loading branch information
linsword13 committed Jul 31, 2024
1 parent 1179cbe commit 631780d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
7 changes: 7 additions & 0 deletions lib/ramble/docs/tutorials/1_hello_world.rst
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,13 @@ Ramble would create an anonymous workspace for you in ``${PWD}/hello_world``
for more information on named and anonymous workspaces, see
:ref:`Ramble workspace documentation<ramble-workspaces>`.

For convenience, the workspace creation and activation can also be combined
in one command with the activate flag (``-a``):

.. code-block:: console
$ ramble workspace create hello_world -a
Configure the Workspace
^^^^^^^^^^^^^^^^^^^^^^^^

Expand Down
4 changes: 1 addition & 3 deletions share/ramble/cloud-build/tutorials/ramble-tutorial-1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,7 @@ steps:
ramble info hostname
ramble workspace create -d hello_world -c /workspace/examples/tutorial_1_config.yaml
ramble workspace activate ./hello_world
ramble workspace create -d hello_world -c /workspace/examples/tutorial_1_config.yaml -a
ramble workspace setup
Expand Down

0 comments on commit 631780d

Please sign in to comment.