Badgerati/Edison

View on GitHub

Showing 215 of 215 total issues

File TestThread.cs has 310 lines of code (exceeds 250 allowed). Consider refactoring.
Open

using Edison.Engine.Contexts;
using Edison.Engine.Repositories.Interfaces;
using Edison.Engine.Utilities.Structures;
using Edison.Framework;
using Edison.Injector;
Severity: Minor
Found in Edison.Engine/Threading/TestThread.cs - About 3 hrs to fix

    Similar blocks of code found in 4 locations. Consider refactoring.
    Open

            [Test]
            [ExpectedException(ExpectedException = typeof(ValidationException), ExpectedMessage = "Assembly is not a valid .dll file: 'dummy/path/to.txt'")]
            public void ValidFileWithInvalidDllForAssemblyTest()
            {
                var dll = "dummy/path/to.txt";
    Severity: Major
    Found in Edison.Engine.Test/Contexts/ContextValidatorTests.cs and 3 other locations - About 3 hrs to fix
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 152..168
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 290..306
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 344..360

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 231.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 4 locations. Consider refactoring.
    Open

            [Test]
            [ExpectedException(ExpectedException = typeof(ValidationException), ExpectedMessage = "File for list of fixtures not found: 'dummy/path/to/file'")]
            public void InvalidFileForFixturesTest()
            {
                var fixture = "this.is.some.fixture";
    Severity: Major
    Found in Edison.Engine.Test/Contexts/ContextValidatorTests.cs and 3 other locations - About 3 hrs to fix
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 134..150
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 152..168
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 344..360

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 231.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 4 locations. Consider refactoring.
    Open

            [Test]
            [ExpectedException(ExpectedException = typeof(ValidationException), ExpectedMessage = "File for list of assemblies not found: 'dummy/path/to/file'")]
            public void InvalidFileForAssemblyTest()
            {
                var dll = "dummy/path/to.dll";
    Severity: Major
    Found in Edison.Engine.Test/Contexts/ContextValidatorTests.cs and 3 other locations - About 3 hrs to fix
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 134..150
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 290..306
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 344..360

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 231.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 4 locations. Consider refactoring.
    Open

            [Test]
            [ExpectedException(ExpectedException = typeof(ValidationException), ExpectedMessage = "File for list of tests not found: 'dummy/path/to/file'")]
            public void InvalidFileForTestsTest()
            {
                var test = "this.is.some.test";
    Severity: Major
    Found in Edison.Engine.Test/Contexts/ContextValidatorTests.cs and 3 other locations - About 3 hrs to fix
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 134..150
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 152..168
    Edison.Engine.Test/Contexts/ContextValidatorTests.cs on lines 290..306

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 231.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void InvalidOutputTypeTest()
            {
                var dll = "dummy/path/to.dll";
    
    
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 444..469

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 227.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void InvalidConsoleOutputTypeTest()
            {
                var dll = "dummy/path/to.dll";
    
    
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 492..517

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 227.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void ValidTestTest()
            {
                var test = "this.is.some.test";
                var dll = "dummy/path/to.dll";
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 275..291

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 220.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void ValidFixtureTest()
            {
                var fixture = "this.is.some.fixture";
                var dll = "dummy/path/to.dll";
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 351..367

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 220.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void InvalidValueForFixtureThreadsTest()
            {
                var dll = "dummy/path/to.dll";
    
    
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 114..138

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 216.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Similar blocks of code found in 2 locations. Consider refactoring.
    Open

            [Test]
            public void InvalidValueForTestThreadsTest()
            {
                var dll = "dummy/path/to.dll";
    
    
    Severity: Major
    Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 3 hrs to fix
    Edison.Console.Test/ParameterParserTests.cs on lines 73..97

    Duplicated Code

    Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

    Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

    When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

    Tuning

    This issue has a mass of 216.

    We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

    The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

    If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

    See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

    Refactorings

    Further Reading

    Method RunThreads has 73 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

            private void RunThreads(Assembly assembly, Exception globalSetupEx)
            {
    
                #region Fetch tests to run
    
    
    Severity: Major
    Found in Edison.Engine/Contexts/EdisonContext.cs - About 2 hrs to fix

      File ContextValidatorTests.cs has 287 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      using Edison.Engine.Contexts;
      using Edison.Engine.Core.Exceptions;
      using Edison.Engine.Repositories.Interfaces;
      using Edison.Engine.Validators;
      using Edison.Injector;
      Severity: Minor
      Found in Edison.Engine.Test/Contexts/ContextValidatorTests.cs - About 2 hrs to fix

        Similar blocks of code found in 2 locations. Consider refactoring.
        Open

                [Test]
                public void ValidConsoleOutputTypeTest()
                {
                    var dll = "dummy/path/to.dll";
        
        
        Severity: Major
        Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 2 hrs to fix
        Edison.Console.Test/ParameterParserTests.cs on lines 475..490

        Duplicated Code

        Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

        Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

        When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

        Tuning

        This issue has a mass of 194.

        We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

        The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

        If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

        See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

        Refactorings

        Further Reading

        Similar blocks of code found in 2 locations. Consider refactoring.
        Open

                [Test]
                public void ValidOutputTypeTest()
                {
                    var dll = "dummy/path/to.dll";
        
        
        Severity: Major
        Found in Edison.Console.Test/ParameterParserTests.cs and 1 other location - About 2 hrs to fix
        Edison.Console.Test/ParameterParserTests.cs on lines 427..442

        Duplicated Code

        Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

        Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

        When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

        Tuning

        This issue has a mass of 194.

        We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

        The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

        If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

        See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

        Refactorings

        Further Reading

        Similar blocks of code found in 3 locations. Consider refactoring.
        Open

                [Test]
                public void ValidSuiteTest()
                {
                    var dll = "dummy/path/to.dll";
        
        
        Severity: Major
        Found in Edison.Console.Test/ParameterParserTests.cs and 2 other locations - About 2 hrs to fix
        Edison.Console.Test/ParameterParserTests.cs on lines 662..677
        Edison.Console.Test/ParameterParserTests.cs on lines 812..827

        Duplicated Code

        Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

        Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

        When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

        Tuning

        This issue has a mass of 191.

        We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

        The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

        If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

        See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

        Refactorings

        Further Reading

        Similar blocks of code found in 3 locations. Consider refactoring.
        Open

                [Test]
                public void ValidSlackTokenTest()
                {
                    var dll = "dummy/path/to.dll";
        
        
        Severity: Major
        Found in Edison.Console.Test/ParameterParserTests.cs and 2 other locations - About 2 hrs to fix
        Edison.Console.Test/ParameterParserTests.cs on lines 625..640
        Edison.Console.Test/ParameterParserTests.cs on lines 812..827

        Duplicated Code

        Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

        Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

        When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

        Tuning

        This issue has a mass of 191.

        We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

        The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

        If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

        See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

        Refactorings

        Further Reading

        Similar blocks of code found in 3 locations. Consider refactoring.
        Open

                [Test]
                public void ValidSolutionConfigTest()
                {
                    var sln = "dummy/path/to.sln";
        
        
        Severity: Major
        Found in Edison.Console.Test/ParameterParserTests.cs and 2 other locations - About 2 hrs to fix
        Edison.Console.Test/ParameterParserTests.cs on lines 625..640
        Edison.Console.Test/ParameterParserTests.cs on lines 662..677

        Duplicated Code

        Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

        Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

        When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

        Tuning

        This issue has a mass of 191.

        We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

        The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

        If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

        See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

        Refactorings

        Further Reading

        Method RunTestCase has 67 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

                private void RunTestCase(MethodInfo test, TestCaseAttribute testCase, int testRepeat, IEnumerable<MethodInfo> setup, IEnumerable<MethodInfo> teardown)
                {
                    var timeTaken = new Stopwatch();
                    var testResult = default(TestResult);
        
        
        Severity: Major
        Found in Edison.Engine/Threading/TestThread.cs - About 2 hrs to fix

          Method Parse has 67 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

                  public static bool Parse(EdisonContext context, string[] args)
                  {
                      if (context == default(EdisonContext))
                      {
                          throw new Exception("No EdisonContext supplied for parsing parameters");
          Severity: Major
          Found in Edison.Console/ParameterParser.cs - About 2 hrs to fix
            Severity
            Category
            Status
            Source
            Language