Sign upLogin

microting/eform-sdk-dotnet

View on GitHub
Star

Showing 2,526 of 2,542 total issues

Identical blocks of code found in 6 locations. Consider refactoring.
Open

            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
Severity: Major
Found in eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.Designer.cs and 5 other locations - About 2 days to fix
eFormCore/Migrations/20211116085744_AddingDoneAtEditable.Designer.cs on lines 445..597
eFormCore/Migrations/20221016081344_AddingIsActiveToLanguage.Designer.cs on lines 445..597
eFormCore/Migrations/20221129082337_AddingReceivedByServerAtToCases.Designer.cs on lines 451..603
eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs on lines 452..604
eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs on lines 452..604

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 1414.

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

Identical blocks of code found in 6 locations. Consider refactoring.
Open

            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
Severity: Major
Found in eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs and 5 other locations - About 2 days to fix
eFormCore/Migrations/20211116085744_AddingDoneAtEditable.Designer.cs on lines 445..597
eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.Designer.cs on lines 445..597
eFormCore/Migrations/20221016081344_AddingIsActiveToLanguage.Designer.cs on lines 445..597
eFormCore/Migrations/20221129082337_AddingReceivedByServerAtToCases.Designer.cs on lines 451..603
eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs on lines 452..604

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 1414.

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

Identical blocks of code found in 6 locations. Consider refactoring.
Open

            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
Severity: Major
Found in eFormCore/Migrations/20221129082337_AddingReceivedByServerAtToCases.Designer.cs and 5 other locations - About 2 days to fix
eFormCore/Migrations/20211116085744_AddingDoneAtEditable.Designer.cs on lines 445..597
eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.Designer.cs on lines 445..597
eFormCore/Migrations/20221016081344_AddingIsActiveToLanguage.Designer.cs on lines 445..597
eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs on lines 452..604
eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs on lines 452..604

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 1414.

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

Identical blocks of code found in 6 locations. Consider refactoring.
Open

            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
Severity: Major
Found in eFormCore/Migrations/20211116085744_AddingDoneAtEditable.Designer.cs and 5 other locations - About 2 days to fix
eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.Designer.cs on lines 445..597
eFormCore/Migrations/20221016081344_AddingIsActiveToLanguage.Designer.cs on lines 445..597
eFormCore/Migrations/20221129082337_AddingReceivedByServerAtToCases.Designer.cs on lines 451..603
eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs on lines 452..604
eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs on lines 452..604

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 1414.

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 ReadFieldValue has a Cognitive Complexity of 154 (exceeds 20 allowed). Consider refactoring.
Open

        private async Task<Models.FieldValue> ReadFieldValue(MicrotingDbContext db, FieldValue reply, Field dbField, bool joinUploadedData,
            Language language)
        {
            string methodName = "SqlController.ReadFieldValue";
            try
Severity: Minor
Found in eFormCore/Infrastructure/SqlController.cs - About 2 days to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

File SqlControllerTestCase.cs has 1107 lines of code (exceeds 250 allowed). Consider refactoring.
Open

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Threading.Tasks;
Severity: Major
Found in eFormSDK.Integration.Case.SqlControllerTests/SqlControllerTestCase.cs - About 2 days to fix

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

                modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckListVersion", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
    Severity: Major
    Found in eFormCore/Migrations/20211014105943_CLAttributes.Designer.cs and 1 other location - About 2 days to fix
    eFormCore/Migrations/20211108111024_AddingIsArchivedToCases.Designer.cs on lines 832..979

    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 1376.

    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

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

                modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckListVersion", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
    Severity: Major
    Found in eFormCore/Migrations/20211108111024_AddingIsArchivedToCases.Designer.cs and 1 other location - About 2 days to fix
    eFormCore/Migrations/20211014105943_CLAttributes.Designer.cs on lines 826..973

    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 1376.

    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

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

                modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
    Severity: Major
    Found in eFormCore/Migrations/20211014105943_CLAttributes.Designer.cs and 1 other location - About 2 days to fix
    eFormCore/Migrations/20211108111024_AddingIsArchivedToCases.Designer.cs on lines 437..583

    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 1360.

    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

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

                modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.CheckList", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
    Severity: Major
    Found in eFormCore/Migrations/20211108111024_AddingIsArchivedToCases.Designer.cs and 1 other location - About 2 days to fix
    eFormCore/Migrations/20211014105943_CLAttributes.Designer.cs on lines 431..577

    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 1360.

    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

    Class SqlController has 131 methods (exceeds 20 allowed). Consider refactoring.
    Open

        public class SqlController : LogWriter
    {
        // var

        private Log _log;
    Severity: Major
    Found in eFormCore/Infrastructure/SqlController.cs - About 2 days to fix

      File SqlControllerTestCaseReadAllShort.cs has 988 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.EntityFrameworkCore;
      Severity: Major
      Found in eFormSDK.Integration.Case.SqlControllerTests/SqlControllerTestCaseReadAllShort.cs - About 2 days to fix

        File 20190116110009_AddingOriginalId.Designer.cs has 971 lines of code (exceeds 250 allowed). Consider refactoring.
        Open

        using System;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Infrastructure;
using Microsoft.EntityFrameworkCore.Metadata;
using Microsoft.EntityFrameworkCore.Migrations;
        Severity: Major
        Found in eFormCore/Migrations/20190116110009_AddingOriginalId.Designer.cs - About 2 days to fix

          File 20180810124416_InitialCreate.Designer.cs has 967 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          using System;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Infrastructure;
using Microsoft.EntityFrameworkCore.Metadata;
using Microsoft.EntityFrameworkCore.Migrations;
          Severity: Major
          Found in eFormCore/Migrations/20180810124416_InitialCreate.Designer.cs - About 2 days to fix

            Method SubChecks has a Cognitive Complexity of 133 (exceeds 20 allowed). Consider refactoring.
            Open

                    private async Task<Element> SubChecks(MicrotingDbContext db, int parentId, Case theCase, Language language)
        {
            string methodName = "SqlController.SubChecks";
            try
            {
            Severity: Minor
            Found in eFormCore/Infrastructure/SqlController.cs - About 2 days to fix

            Cognitive Complexity

            Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

            A method's cognitive complexity is based on a few simple rules:

            • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
            • Code is considered more complex for each "break in the linear flow of the code"
            • Code is considered more complex when "flow breaking structures are nested"

            Further reading

            Method Up has 474 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

                    protected override void Up(MigrationBuilder migrationBuilder)
        {
            //Setup for SQL Server Provider
           
            string autoIDGenStrategy = "SqlServer:ValueGenerationStrategy";
            Severity: Major
            Found in eFormCore/Migrations/20190509074123_RefactoringidtoId.cs - About 2 days to fix

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

                              [Test]
                public async Task question_Update_DoesUpdate_ValidDisplayTrue()
                {
                    // Arrange
                    string name = Guid.NewGuid().ToString();
              Severity: Major
              Found in eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs and 27 other locations - About 2 days to fix
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1912..1996
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1998..2083
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2087..2171
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2259..2343
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2345..2429
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2430..2514
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2515..2599
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2600..2684
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2685..2769
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2770..2854
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2855..2939
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2940..3024
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3025..3109
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3110..3194
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3199..3284
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3286..3371
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3373..3458
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3460..3545
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3546..3631
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3632..3717
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3718..3803
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3804..3889
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3890..3975
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3976..4061
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4062..4147
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4148..4234
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4235..4319

              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 1165.

              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 28 locations. Consider refactoring.
              Open

                              [Test]
                public async Task question_Update_DoesUpdate_ValidDisplayBackButtonEnabledImageTrue()
                {
                    // Arrange
                    string name = Guid.NewGuid().ToString();
              Severity: Major
              Found in eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs and 27 other locations - About 2 days to fix
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1912..1996
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1998..2083
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2087..2171
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2173..2257
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2259..2343
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2345..2429
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2430..2514
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2515..2599
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2600..2684
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2685..2769
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2770..2854
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2855..2939
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2940..3024
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3025..3109
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3199..3284
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3286..3371
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3373..3458
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3460..3545
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3546..3631
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3632..3717
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3718..3803
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3804..3889
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3890..3975
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3976..4061
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4062..4147
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4148..4234
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4235..4319

              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 1165.

              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 28 locations. Consider refactoring.
              Open

                          [Test]
            public async Task question_Update_DoesUpdate_ValidDisplayTrue_QSFalse()
            {
                // Arrange
                string name = Guid.NewGuid().ToString();
              Severity: Major
              Found in eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs and 27 other locations - About 2 days to fix
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1912..1996
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1998..2083
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2087..2171
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2173..2257
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2259..2343
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2345..2429
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2430..2514
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2515..2599
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2600..2684
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2685..2769
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2770..2854
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2855..2939
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2940..3024
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3025..3109
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3110..3194
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3199..3284
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3373..3458
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3460..3545
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3546..3631
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3632..3717
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3718..3803
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3804..3889
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3890..3975
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3976..4061
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4062..4147
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4148..4234
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4235..4319

              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 1165.

              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 28 locations. Consider refactoring.
              Open

                                          [Test]
            public async Task question_Update_DoesUpdate_PrioritisedValidDisplayTrue_QSFalse()
            {
                // Arrange
                string name = Guid.NewGuid().ToString();
              Severity: Major
              Found in eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs and 27 other locations - About 2 days to fix
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1912..1996
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 1998..2083
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2087..2171
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2173..2257
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2259..2343
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2345..2429
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2430..2514
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2515..2599
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2600..2684
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2685..2769
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2770..2854
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2855..2939
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 2940..3024
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3025..3109
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3110..3194
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3199..3284
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3286..3371
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3373..3458
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3460..3545
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3632..3717
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3718..3803
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3804..3889
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3890..3975
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 3976..4061
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4062..4147
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4148..4234
              eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs on lines 4235..4319

              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 1165.

              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

              Severity
              Category
              Status
              Source
              Language