Project

General

Profile

action #59354

[qam][newt][tcsh] standalone test for tcsh due to regression

Added by brhavel almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
Enhancement to existing tests
Target version:
-
Start date:
2019-11-12
Due date:
2019-11-22
% Done:

100%

Estimated time:
Difficulty:
easy

Description

There was a regression in tcsh causing issue to customer and they request regression test to avoid such issue in the future. There are tests running during the build, however, they do not capture this particular problem as user needs to use the shell interactively. Possible issue can be reproduced as follows (from the package maintainer):

The regression can only be seen if tcsh is interactively used. That is simply login as a tcsh user, that has a /usr/bin/tcsh or /bin/tcsh as login shell. If the prompt becomes broken and does not work anymore you had caught this bug. That is things like

ls i-d /*

(should return the directory and file list below /) or

echo ~

(should return the current home directory) do not work anymore due a broken tcsh.

reported bug which can be used as source of info for the test:
https://bugzilla.suse.com/show_bug.cgi?id=1154877

History

#1 Updated by brhavel almost 2 years ago

  • Priority changed from Normal to Urgent

#2 Updated by slemke almost 2 years ago

  • Assignee set to slemke

#3 Updated by okurz almost 2 years ago

http://github.com/os-autoinst/os-autoinst-distri-opensuse/tree/master/tests/console/shells.pm already spawns tcsh. Should be easy to extend it. I do not see much value in just adding a regression test for the specific bug though. Maybe you want to extend the test further therefore :)

#4 Updated by brhavel almost 2 years ago

I agree with Oliver. @Sergio, can you please extend respective test accordingly? Thank you

#5 Updated by WernerFink almost 2 years ago

The test ls i-d /* should be read as ls -d /*

#6 Updated by slemke almost 2 years ago

No problem, I have id done in a new class already, but should not be problematic to extract to the existing one:
See: http://deathstar.suse.cz/tests/1997

#7 Updated by slemke almost 2 years ago

ok, added a function containing this procedures:
eg: http://deathstar.suse.cz/tests/2034

will create a PR and lets see.

#9 Updated by slemke almost 2 years ago

  • Category changed from New test to Enhancement to existing tests
  • Status changed from New to Resolved
  • % Done changed from 0 to 100
  • Difficulty set to easy

Also available in: Atom PDF