Project

General

Profile

Actions

action #59354

closed

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

Added by brhavel about 5 years ago. Updated about 5 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

Actions #1

Updated by brhavel about 5 years ago

  • Priority changed from Normal to Urgent
Actions #2

Updated by slemke about 5 years ago

  • Assignee set to slemke
Actions #3

Updated by okurz about 5 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 :)

Actions #4

Updated by brhavel about 5 years ago

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

Actions #5

Updated by WernerFink about 5 years ago

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

Actions #6

Updated by slemke about 5 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

Actions #7

Updated by slemke about 5 years ago

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

will create a PR and lets see.

Actions #9

Updated by slemke about 5 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
Actions

Also available in: Atom PDF