Adding unit tests to verify the fix.

Author: Keboo

MaterialDesignThemes.Wpf.Tests/ClockTests.cs

@@ -0,0 +1,63 @@
+using System.Linq;
+using System.Windows.Controls;
+using Xunit;
+
+namespace MaterialDesignThemes.Wpf.Tests
+{
+    public class ClockTests
+    {
+        [StaFact(Timeout = 500)]
+        public void CanGenerateHoursButtons()
+        {
+            var clock = new Clock();
+            clock.ApplyDefaultStyle();
+
+            Canvas hoursCanvas = clock.FindVisualChild<Canvas>(Clock.HoursCanvasPartName);
+            var buttonContents = hoursCanvas.GetVisualChildren<ClockItemButton>().Select(x => x.Content.ToString());
+
+            var expected = Enumerable.Range(1, 12).Select(x => $"{x:0}");
+            Assert.Equal(expected.OrderBy(x => x), buttonContents.OrderBy(x => x));
+        }
+
+        [StaFact(Timeout = 500)]
+        public void CanGenerateHoursButtonsWith24Hours()
+        {
+            var clock = new Clock {Is24Hours = true};
+            clock.ApplyDefaultStyle();
+
+            Canvas hoursCanvas = clock.FindVisualChild<Canvas>(Clock.HoursCanvasPartName);
+            var buttonContents = hoursCanvas.GetVisualChildren<ClockItemButton>().Select(x => x.Content.ToString());
+
+            var expected = Enumerable.Range(13, 11).Select(x => $"{x:00}")
+                .Concat(new [] {"00"})
+                .Concat(Enumerable.Range(1, 12).Select(x => $"{x:#}"));
+            Assert.Equal(expected.OrderBy(x => x), buttonContents.OrderBy(x => x));
+        }
+
+        [StaFact(Timeout = 500)]
+        public void CanGenerateMinuteButtons()
+        {
+            var clock = new Clock();
+            clock.ApplyDefaultStyle();
+
+            Canvas minutesCanvas = clock.FindVisualChild<Canvas>(Clock.MinutesCanvasPartName);
+            var buttonContents = minutesCanvas.GetVisualChildren<ClockItemButton>().Select(x => x.Content.ToString());
+
+            var expected = Enumerable.Range(0, 60).Select(x => $"{x:0}");
+            Assert.Equal(expected.OrderBy(x => x), buttonContents.OrderBy(x => x));
+        }
+
+        [StaFact(Timeout = 500)]
+        public void CanGenerateSecondsButtons()
+        {
+            var clock = new Clock();
+            clock.ApplyDefaultStyle();
+
+            Canvas secondsCanvas = clock.FindVisualChild<Canvas>(Clock.SecondsCanvasPartName);
+            var buttonContents = secondsCanvas.GetVisualChildren<ClockItemButton>().Select(x => x.Content.ToString());
+
+            var expected = Enumerable.Range(0, 60).Select(x => $"{x:0}");
+            Assert.Equal(expected.OrderBy(x => x), buttonContents.OrderBy(x => x));
+        }
+    }
+}

MaterialDesignThemes.Wpf.Tests/MaterialDesignThemes.Wpf.Tests.csproj

@@ -12,6 +12,7 @@
     <AssemblyName>MaterialDesignThemes.Wpf.Tests</AssemblyName>
     <TargetFrameworkVersion>v4.5.2</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
+    <LangVersion>latest</LangVersion>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <DebugSymbols>true</DebugSymbols>
@@ -29,6 +30,7 @@
     <DefineConstants>TRACE</DefineConstants>
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
+    <LangVersion>latest</LangVersion>
   </PropertyGroup>
   <ItemGroup>
     <Reference Include="PresentationCore" />
@@ -43,9 +45,12 @@
     <Reference Include="System.Xml" />
   </ItemGroup>
   <ItemGroup>
+    <Compile Include="ClockTests.cs" />
     <Compile Include="DialogHostTests.cs" />
+    <Compile Include="MdixHelper.cs" />
     <Compile Include="PaletteHelperFixture.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
+    <Compile Include="VisualTreeHelper.cs" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\MaterialDesignThemes.Wpf\MaterialDesignThemes.Wpf.csproj">

MaterialDesignThemes.Wpf.Tests/MdixHelper.cs

@@ -0,0 +1,57 @@
+using System;
+using System.IO.Packaging;
+using System.Windows;
+using System.Windows.Controls;
+using Xunit;
+
+namespace MaterialDesignThemes.Wpf.Tests
+{
+    public static class MdixHelper
+    {
+        static MdixHelper()
+        {
+            var _ = Application.Current;
+
+            _defaultResourceDictionary = new Lazy<ResourceDictionary>(() => GetThemeResourceDictionary("Defaults"));
+            _genericResourceDictionary = new Lazy<ResourceDictionary>(GetGenericResourceDictionary);
+        }
+
+        private static readonly Lazy<ResourceDictionary> _defaultResourceDictionary;
+        private static ResourceDictionary DefaultResourceDictionary => _defaultResourceDictionary.Value;
+
+        private static readonly Lazy<ResourceDictionary> _genericResourceDictionary;
+        private static ResourceDictionary GenericResourceDictionary => _genericResourceDictionary.Value;
+
+        public static void ApplyDefaultStyle<T>(this T control) where T : FrameworkElement
+        {
+            Style style = GetDefaultStyle<T>();
+            Assert.True(style != null, $"Could not find default style for control type {typeof(T).FullName}");
+            control.Style = style;
+            Assert.True(control.ApplyTemplate(), "Failed to apply template ");
+        }
+
+        private static Style GetDefaultStyle<T>() where T : FrameworkElement
+        {
+            return DefaultResourceDictionary[typeof(T)] as Style ??
+                   GenericResourceDictionary[typeof(T)] as Style;
+        }
+
+        private static ResourceDictionary GetThemeResourceDictionary(string name)
+        {
+            var uri = new Uri(
+                $"/MaterialDesignThemes.Wpf;component/Themes/MaterialDesignTheme.{name}.xaml", UriKind.Relative);
+            return new ResourceDictionary {
+                Source = uri
+            };
+        }
+
+        private static ResourceDictionary GetGenericResourceDictionary()
+        {
+            var uri = new Uri(
+                $"/MaterialDesignThemes.Wpf;component/Themes/Generic.xaml", UriKind.Relative);
+            return new ResourceDictionary {
+                Source = uri
+            };
+        }
+    }
+}

MaterialDesignThemes.Wpf.Tests/VisualTreeHelper.cs

@@ -0,0 +1,55 @@
+using System;
+using System.Collections;
+using System.Collections.Generic;
+using System.Linq;
+using System.Windows;
+
+using WPFVisualTreeHelper= System.Windows.Media.VisualTreeHelper;
+
+namespace MaterialDesignThemes.Wpf.Tests
+{
+    public static class VisualTreeHelper
+    {
+        public static T FindVisualChild<T>(this DependencyObject source, string name) where T : FrameworkElement
+        {
+            return GetAllVisualChildren(source).OfType<T>().Single(x => x.Name == name);
+        }
+
+        public static IEnumerable<T> GetVisualChildren<T>(this DependencyObject source) where T : DependencyObject
+        {
+            if (source == null) throw new ArgumentNullException(nameof(source));
+
+            return GetVisualChildrenImplementation();
+
+            IEnumerable<T> GetVisualChildrenImplementation()
+            {
+                int count = WPFVisualTreeHelper.GetChildrenCount(source);
+                for (int i = 0; i < count; i++)
+                {
+                    if (WPFVisualTreeHelper.GetChild(source, i) is T child)
+                    {
+                        yield return child;
+                    }
+                }
+            }
+        }
+
+        private static IEnumerable<DependencyObject> GetAllVisualChildren(DependencyObject source)
+        {
+            var stack = new Queue<DependencyObject>();
+            stack.Enqueue(source);
+
+            while (stack.Any())
+            {
+                DependencyObject current = stack.Dequeue();
+                int childCount = WPFVisualTreeHelper.GetChildrenCount(current);
+                for (int i = 0; i < childCount; i++)
+                {
+                    var child = WPFVisualTreeHelper.GetChild(current, i);
+                    yield return child;
+                    stack.Enqueue(child);
+                }
+            }
+        }
+    }
+}

MaterialDesignThemes.Wpf/Converters/ClockItemIsCheckedConverter.cs

@@ -4,81 +4,79 @@
 
 namespace MaterialDesignThemes.Wpf.Converters
 {
-	internal class ClockItemIsCheckedConverter : IValueConverter
-	{
-		private readonly Func<DateTime> _currentTimeGetter;
-		private readonly ClockDisplayMode _displayMode;
-	    private readonly bool _is24Hours;
+    internal class ClockItemIsCheckedConverter : IValueConverter
+    {
+        private readonly Func<DateTime> _currentTimeGetter;
+        private readonly ClockDisplayMode _displayMode;
+        private readonly bool _is24Hours;
 
-		public ClockItemIsCheckedConverter(Func<DateTime> currentTimeGetter, ClockDisplayMode displayMode, bool is24Hours)
-		{
-			if (currentTimeGetter == null) throw new ArgumentNullException(nameof(currentTimeGetter));
+        public ClockItemIsCheckedConverter(Func<DateTime> currentTimeGetter, ClockDisplayMode displayMode, bool is24Hours)
+        {
+            _currentTimeGetter = currentTimeGetter ?? throw new ArgumentNullException(nameof(currentTimeGetter));
+            _displayMode = displayMode;
+            _is24Hours = is24Hours;
+        }
 
-			_currentTimeGetter = currentTimeGetter;
-			_displayMode = displayMode;
-		    _is24Hours = is24Hours;
-		}
-
-		public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
-		{
-			var dateTime = (DateTime) value;
-			var i = (int) parameter;
+        public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
+        {
+            var dateTime = (DateTime)value;
+            var i = (int)parameter;
 
             int converted = 0;
-            if (_displayMode == ClockDisplayMode.Hours )
+            if (_displayMode == ClockDisplayMode.Hours)
                 converted = MassageHour(dateTime.Hour, _is24Hours);
             else if (_displayMode == ClockDisplayMode.Minutes)
                 converted = MassageMinuteSecond(dateTime.Minute);
             else
                 converted = MassageMinuteSecond(dateTime.Second);
             return converted == i;
-		}
+        }
 
-		public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
-		{
-			if ((bool)value != true) return Binding.DoNothing;
+        public object ConvertBack(object value, Type targetType, object parameter, CultureInfo culture)
+        {
+            if ((bool)value != true) return Binding.DoNothing;
 
-			var currentTime = _currentTimeGetter();
+            var currentTime = _currentTimeGetter();
 
             return new DateTime(
                 currentTime.Year, currentTime.Month, currentTime.Day,
                 (_displayMode == ClockDisplayMode.Hours) ? ReverseMassageHour((int)parameter, currentTime, _is24Hours) : currentTime.Hour,
                 (_displayMode == ClockDisplayMode.Minutes) ? ReverseMassageMinuteSecond((int)parameter) : currentTime.Minute,
                 (_displayMode == ClockDisplayMode.Seconds) ? ReverseMassageMinuteSecond((int)parameter) : currentTime.Second);
-		}
+        }
 
-		private static int MassageHour(int val, bool is24Hours)
-		{
-		    if (is24Hours)
-		    {		        
-		        return val == 0 ? 24 : val;
-		    }
+        private static int MassageHour(int val, bool is24Hours)
+        {
+            if (is24Hours)
+            {
+                return val == 0 ? 24 : val;
+            }
 
-			if (val == 0) return 12;
-			if (val > 12) return val - 12;
-			return val;
-		}
+            if (val == 0) return 12;
+            if (val > 12) return val - 12;
+            return val;
+        }
 
-		private static int MassageMinuteSecond(int val)
-		{
-			return val == 0 ? 60 : val;
-		}
+        private static int MassageMinuteSecond(int val)
+        {
+            return val == 0 ? 60 : val;
+        }
 
-		private static int ReverseMassageHour(int val, DateTime currentTime, bool is24Hours)
-		{
-		    if (is24Hours)
-		    {
-		        return val == 24 ? 0 : val;
-		    }
+        private static int ReverseMassageHour(int val, DateTime currentTime, bool is24Hours)
+        {
+            if (is24Hours)
+            {
+                return val == 24 ? 0 : val;
+            }
 
-			return currentTime.Hour < 12 
-				? (val == 12 ? 0 : val)
-				: (val == 12 ? 12 : val + 12);
-		}
+            return currentTime.Hour < 12
+                ? (val == 12 ? 0 : val)
+                : (val == 12 ? 12 : val + 12);
+        }
 
-		private static int ReverseMassageMinuteSecond(int val)
-		{
-			return val == 60 ? 0 : val;
-		}
-	}
+        private static int ReverseMassageMinuteSecond(int val)
+        {
+            return val == 60 ? 0 : val;
+        }
+    }
 }