Find unused code in a Rails app

How do I find what code is and isn't being run in production ?

The app is well-tested, but there's a lot of tests that test unused code. Hence they get coverage when running tests... I'd like to refactor and clean up this mess, it keeps wasting my time. I have a lot of background jobs, this is why I'd like the production env to guide me. Running at heroku I can spin up dynos to compensate any performance impacts from the profiler.

Related question https://stackoverflow.com/q/2958443/252799 not helpful.

Bonus: metrics to show how often a line of code is run. Don't know why I want it, but I do! :)

Under normal circumstances the approach would be to use your test data for code coverage, but as you say you have parts of your code that are tested but are not used on the production app, you could do something slightly different.

Just for clarity first: Don't trust automatic tools. They will only show you results for things you actively test, nothing more.

With the disclaimer behind us, I propose you use a code coverage tool (like rcov or simplecov for Ruby 1.9) on your production app and measure the code paths that are actually used by your users. While these tools were originally designed for measuring test coverage, you could also use them for production coverage

Under the assumption that during the test time-frame all relevant code paths are visited, you can remove the rest. Unfortunately, this assumption will most probably not fully hold. So you will still have to apply your knowledge of the app and its inner workings when removing parts. This is even more important when removing declarative parts (like model references) as those are often not directly run but only used for configuring other parts of the system.

Another approach which could be combined with the above is to try to refactor your app into distinguished features that you can turn on and off. Then you can turn features that are suspected to be unused off and check if nobody complains :)

And as a final note: you won't find a magic tool to do your full analysis. That's because no tool can know whether a certain piece of code is used by actual users or not. The only thing that tools can do is create (more or less) static reachability graphs, telling you if your code is somehow called from a certain point. With a dynamic language like Ruby even this is rather hard to achieve, as static analysis doesn't bring much insight in the face of meta-programming or dynamic calls that are heavily used in a rails context. So some tools actually run your code or try to get insight from test coverage. But there is definitely no magic spell.

So given the high internal (mostly hidden) complexity of a rails application, you will not get around to do most of the analysis by hand. The best advice would probably be to try to modularize your app and turn off certain modules to test f they are not used. This can be supported by proper integration tests.

Maybe you can try to use rails_best_practices to check unused methods and class.

Here it is in the github: https://github.com/railsbp/rails_best_practices .

Put 'gem "rails_best_practices" ' in your Gemfile and then run rails_best_practices . to generate configuration file

Checkout the coverband gem, it does what you exactly what are you searching.

I had the same problem and after exploring some alternatives I realized that I have all the info available out of the box - log files. Our log format is as follows

Dec 18 03:10:41 ip-xx-xx-xx-xx appname-p[7776]:   Processing by MyController#show as HTML

So I created a simple script to parse this info

zfgrep Processing production.log*.gz |awk '{print $8}' > ~/tmp/action

sort  ~/tmp/action | uniq -c |sort -g -r > ~/tmp/histogram

Which produced results of how often an given controller#action was accessed.

4394886 MyController#index
3237203 MyController#show
1644765 MyController#edit

Next step is to compare it to the list of all controller#action pair in the app (using rake routes output or can do the same script for testing suite)

You got already the idea to mark suspicious methods as private (what will maybe break your application).

A small variation I did in the past: Add a small piece code to all suspicious methods to log it. In my case it was a user popup "You called a obsolete function - if you really need please contact the IT". After one year we had a good overview what was really used (it was a business application and there where functions needed only once a year).

In your case you should only log the usage. Everything what is not logged after a reasonable period is unused.

I'm not very familiar with Ruby and RoR, but what I'd suggest some crazy guess:

• add :after_filter method wich logs name of previous called method(grab it from call stack) to file
• deploy this to production
• wait for a while
• remove all methods that are not in log.

p.s. probably solution with Alt+F7 in NetBeans or RubyMine is much better :)

Metaprogramming

Object#method_missing

override Object#method_missing. Inside, log the calling Class and method, asynchronously, to a data store. Then manually call the original method with the proper arguments, based on the arguments passed to method_missing.

Object tree

Then compare the data in the data store to the contents of the application's object tree.

disclaimer: This will surely require significant performance and resource consideration. Also, it will take a little tinkering to get that to work, but theoretically it should work. I'll leave it as an exercise to the original poster to implement it. ;)

Have you tried creating a test suite using something like sahi you could then record all your user journies using this and tie those tests to rcov or something similar.

You do have to ensure you have all user journies but after that you can look at what rcov spits out and at least start to prune out stuff that is obviously never covered.

This isn't a very proactive approach, but I've often used results gathered from New Relic to see if something I suspected as being unused had been called in production anytime in the past month or so. The apps I've used this on have been pretty small though, and its decently expensive for larger applications.

I've never used it myself, but this post about the laser gem seems to talk about solving your exact problem.

It is not the perfect solution, but for example in NetBeans you can find usages of the methods by right click on them (or press Alt+F7).
So if method is unused, you will see it.

mark suspicious methods as private. If that does not break the code, check if the methods are used inside the class. then you can delete things