Test help output in `rich-click` CLI is robust to lazy loading of modules.

[dwreeves]

Test help output in `rich-click` CLI is robust to lazy loading of modules.

[dwreeves]

Ah it gets worse. celery beat --help:

Usage: celery beat [OPTIONS]

  Start the beat periodic task scheduler.

Beat Options:
  --detach                        Detach and run in the background as a
                                  daemon.
  -s, --schedule TEXT             Path to the schedule database.  Defaults to
                                  `celerybeat-schedule`.The extension '.db'
                                  may be appended to the filename.
  -S, --scheduler TEXT            Scheduler class to use.
  --max-interval INTEGER          Max seconds to sleep between schedule
                                  iterations.
  -l, --loglevel [DEBUG|INFO|WARNING|ERROR|CRITICAL|FATAL]
                                  Logging level.

Daemonization Options:
  -f, --logfile TEXT  Log destination; defaults to stderr
  --pidfile TEXT
  --uid TEXT
  --gid TEXT
  --umask TEXT
  --executable TEXT

Options:
  --help  Show this message and exit.

rich-click celery beat --help:

                                                                                                    
 Usage: celery beat [OPTIONS]                                                                       
                                                                                                    
 Start the beat periodic task scheduler.                                                            
                                                                                                    
Beat Options:

  --detach
                        
Detach and run in the background as a daemon.

  -s, --schedule TEXT
             
Path to the schedule database.  Defaults to `celerybeat-

                                  schedule`.The extension '.db' may be appended to the filename.

  -S, --scheduler TEXT
            
Scheduler class to use.

  --max-interval INTEGER
          
Max seconds to sleep between schedule iterations.

  -l, --loglevel [DEBUG|INFO|WARNING|ERROR|CRITICAL|FATAL]


                                  
Logging level.

Daemonization Options:

  -f, --logfile TEXT
  
Log destination; defaults to stderr

  --pidfile TEXT


  --uid TEXT


  --gid TEXT


  --umask TEXT


  --executable TEXT


Options:

  --help
  
Show this message and exit.

Note this is specific to 1.8.0dev. 1.7.3 does not have this issue. I have a feeling why that is the case, but I will have to look deeper.

[dwreeves]

Figured out what's going on pretty quickly with the subcommand and I'll have a fix for that, but the main command I will have to dig into.

[dwreeves]

Ahhh, figured it out. TLDR, this is a bug in 1.8.0dev, and it arises due to the order of how Click stuff is loaded now with the lazy-loading.

This is entirely fixable, and in that sense it's not a big deal.

What is a big deal is this is a big pain to safeguard against in tests, since the tests all share the same sys.modules and the same click module, normally invoked test cases of click Groups will contain false passes. I need to think about how to guard against this!