Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test of laguage name. #1026

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Aug 14, 2023

  1. Add test of laguage name.

    The current generated tests from the data folder check for the
    report values but not if the the correct laguage was identified.
    The problem here is if the wrong language is identified but by luck
    the counts match, the tests wont catch that.
    
    Example:
    
    lets say we have a zsh.zsh file that tokei thinks is a bash script.
    Given how simillar they are, the counts will likely be correct even
    if the extesion is misenturpreted, but the report produced by tokei
    will put the counts under the wrong laguage lable.
    
    A more tricky example is that of ruby_env which relys on the shebang
    and the `/usr/bin/env ruby` which is less obvious then some other forms
    html
    glazari committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    5bc9cfb View commit details
    Browse the repository at this point in the history