remove all spec views on destroy controller

[Naokimi]

Actions:

1. rails g controller Authors index show
2. rails d controller Authors

Current
ON GENERATE
Rails generates controller and view files
Rspec generates controller and view files
ON DESTROY
Rails destroys controller and view files
Rspec destroys only controller

Proposal
ON DESTROY
Rails destroys controller and view files
Rspec destroys controller and view files

[pirj]

Awesome! Can you please add specs for this change?

[Naokimi]

run_generator %w[posts], behavior: :revoke isn't working and I can't figure out why (it apparently doesn't run, but it also doesn't raise any errors)
they use it for rails: https://github.com/rails/rails/blob/main/railties/test/generators/task_generator_test.rb#L23

EDIT:
sample test:

    describe 'are destroyed' do
      before do
        run_generator %w[posts index]
        run_generator %w[posts], behavior: :revoke
      end
      describe 'index.html.erb' do
        subject { file('spec/views/posts/index.html.erb_spec.rb') }
        it { is_expected.not_to exist }
      end
    end

EDIT2:
The second generator in the before block from the sample test doesn't run. Still haven't figured out why

[Naokimi]

@pirj found a workaround, ready to be reviewed again

[benoittgt]

Hello @Naokimi and thanks for you PR.

Tested locally and it is looks good to me.

[JonRowe]

Do we have this problem with other generators? Do we want to ensure we're consistent?

[Naokimi]

seems like other generators work as intended
just controller one had a guard clause that would block expected behaviour