Badgerati/Edison

Class ReflectionRepository has 21 methods (exceeds 20 allowed). Consider refactoring.

Open

    [InjectionInterface(typeof(IReflectionRepository))]
    public class ReflectionRepository : IReflectionRepository
    {

        #region Constants

Method HasValidCategories has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public bool HasValidCategories(MemberInfo member, IList<string> includedCategories, IList<string> excludedCategories, bool testFixtureDefault = true)
        {
            // if no categories are passed, just return true
            if (EnumerableHelper.IsNullOrEmpty(includedCategories) && EnumerableHelper.IsNullOrEmpty(excludedCategories))
            {

Method GetMethods has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        public IEnumerable<MethodInfo> GetMethods<T>(Type type,
            IList<string> includedCategories = default(List<string>),
            IList<string> excludedCategories = default(List<string>),
            IList<string> tests = default(List<string>),
            string suite = null) where T : Attribute

Avoid too many return statements within this method.

Open

                return testFixtureDefault;

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

                return fixtureCategories.Any() && HasValidCategories(fixture, includedCategories, excludedCategories, false);

Avoid too many return statements within this method.

Open

            return true;

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

Open

            if (!EnumerableHelper.IsNullOrEmpty(excludedCategories) && categories.Any(c => excludedCategories.Any(e => e.Equals(c.Name, StringComparison.InvariantCultureIgnoreCase))))
            {
                return false;
            }

• Read up
  Read up

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

Open

            if (!EnumerableHelper.IsNullOrEmpty(includedCategories) && categories.Any(c => includedCategories.Any(i => i.Equals(c.Name, StringComparison.InvariantCultureIgnoreCase))))
            {
                return true;
            }

• Read up
  Read up

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

Open

        public ExpectedExceptionAttribute GetExpectedException(MethodInfo method)
        {
            var exceptionAttribute = method.GetCustomAttributes().OfType<ExpectedExceptionAttribute>();

            return EnumerableHelper.IsNullOrEmpty(exceptionAttribute)

• Read up
  Read up

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

Open

        public SlackAttribute GetSlackChannel(MethodInfo method)
        {
            var slackAttribute = method.GetCustomAttributes().OfType<SlackAttribute>();

            return EnumerableHelper.IsNullOrEmpty(slackAttribute)

• Read up
  Read up