#2054 beaker: fix expected error msg after project fork
Closed 2 years ago by praiskup. Opened 2 years ago by schlupov.
copr/ schlupov/copr fix_fork_beaker_test  into  main

@@ -50,8 +50,8 @@ 

          # attempt to fork into existing project

          OUTPUT=`mktemp`

          rlRun "copr-cli fork ${NAME_PREFIX}Project10 ${NAME_PREFIX}Project10Fork &> $OUTPUT" 1

-         rlAssertEquals "Error existing project" `grep -r 'Error: You are about to fork into existing project' $OUTPUT |wc -l` 1

-         rlAssertEquals "Use --confirm" `grep -r 'Please use --confirm if you really want to do this' $OUTPUT |wc -l` 1

+         rlAssertEquals "Error existing project" `grep -r '\- You are about to fork into existing project' $OUTPUT |wc -l` 1

+         rlAssertEquals "Use --confirm" `grep -r '\- Please use --confirm if you really want to do this' $OUTPUT |wc -l` 1

          rlRun "copr-cli list-builds ${NAME_PREFIX}Project10Fork > $OUTPUT"

          rlAssertEquals "No build was forked" `cat $OUTPUT | wc -l` 1

  

@@ -379,8 +379,8 @@ 

          # attempt to fork into existing project

          OUTPUT=`mktemp`

          rlRun "copr-cli fork ${NAME_PREFIX}Project10 ${NAME_PREFIX}Project10Fork &> $OUTPUT" 1

-         rlAssertEquals "Error existing project" `grep -r 'Error: You are about to fork into existing project' $OUTPUT |wc -l` 1

-         rlAssertEquals "Use --confirm" `grep -r 'Please use --confirm if you really want to do this' $OUTPUT |wc -l` 1

+         rlAssertEquals "Error existing project" `grep -r '\- You are about to fork into existing project' $OUTPUT |wc -l` 1

+         rlAssertEquals "Use --confirm" `grep -r '\- Please use --confirm if you really want to do this' $OUTPUT |wc -l` 1

  

          # fork into existing project

          OUTPUT=`mktemp`

Metadata Update from @schlupov:
- Pull-request tagged with: release-blocker

2 years ago

Build succeeded.

Fixes: #2053

There are two problems to be fixed, this one seems to address only one of them.

Fixes: #2053

There are two problems to be fixed, this one seems to address only one of them.

https://pagure.io/copr/copr/issue/2053#comment-777284

rebased onto a526128

2 years ago

Build succeeded.

+1 for this part, merged as 3669fee
The second part still needs a separate fix.

Pull-Request has been closed by praiskup

2 years ago