Added a option "mom_on_server"=<boolean> to existing PTL requirement decorator

Developers
1

Hi All,
Please review design document PBS Pro Confluence
Please do let me know of any comments or suggestions.
Thanks and Regards,
Birbal

2

PTL requirement decorator doesn’t has a option to specify test requirement "mom should be on server.
Present requirement decorator option “no_mom_on_server” is used only to specify mom should not be on server by setting it’s value to True and it’s by default value is False that mean mom can or can not be on server.

3

Thanks @birbalsain for adding mom_on_server requirement to the decorator. Looks good to me.

4

Thanks for the design, looks good to me.

5

Instead of the log message “mom on server” when the test is skipped, can the message please be clearer. After all “mom on server” could mean two different things:

  1. skipping the test because there is a mom on the server
  2. skipping the test because the test needs a mom on the server

How about “test needs a mom on the server node”?

6

I think this message is ok because we get skip message as “what requirement is needed”.

  1. If requirement is “no_mom_on_server” and doesn’t match then skip message is “no mom on server”
    means no mom on server is needed.
    2.like above , If requirement is “mom_on_server” and doesn’t match then skip message is “mom on server”