diff --git a/.vscode/launch.json b/.vscode/launch.json new file mode 100644 index 0000000000..99d196ace7 --- /dev/null +++ b/.vscode/launch.json @@ -0,0 +1,23 @@ +{ + "version": "0.2.0", + "configurations": [ + { + "name": ".NET Core Launch (console)", + "type": "coreclr", + "request": "launch", + "preLaunchTask": "build", + "program": "${workspaceRoot}\\samples\\ControlCatalog.NetCore\\bin\\Debug\\netcoreapp1.1\\ControlCatalog.NetCore.dll", + "args": [], + "cwd": "${workspaceRoot}\\samples\\ControlCatalog.NetCore", + "console": "internalConsole", + "stopAtEntry": false, + "internalConsoleOptions": "openOnSessionStart" + }, + { + "name": ".NET Core Attach", + "type": "coreclr", + "request": "attach", + "processId": "${command:pickProcess}" + } + ] +} \ No newline at end of file diff --git a/.vscode/tasks.json b/.vscode/tasks.json new file mode 100644 index 0000000000..27f8776c5f --- /dev/null +++ b/.vscode/tasks.json @@ -0,0 +1,16 @@ +{ + "version": "0.1.0", + "command": "dotnet", + "isShellCommand": true, + "args": [], + "tasks": [ + { + "taskName": "build", + "args": [ + "${workspaceRoot}\\samples\\ControlCatalog.NetCore\\ControlCatalog.NetCore.csproj" + ], + "isBuildCommand": true, + "problemMatcher": "$msCompile" + } + ] +} \ No newline at end of file diff --git a/Avalonia.v3.ncrunchsolution b/Avalonia.v3.ncrunchsolution index c2c454eae1..1b5b0c8930 100644 --- a/Avalonia.v3.ncrunchsolution +++ b/Avalonia.v3.ncrunchsolution @@ -3,7 +3,7 @@ tests\TestFiles\**.* - False + True .ncrunch True diff --git a/src/Avalonia.Base/IPriorityValueOwner.cs b/src/Avalonia.Base/IPriorityValueOwner.cs index 57f98c0717..aeec720920 100644 --- a/src/Avalonia.Base/IPriorityValueOwner.cs +++ b/src/Avalonia.Base/IPriorityValueOwner.cs @@ -25,5 +25,10 @@ namespace Avalonia /// The source of the change. /// The notification. void BindingNotificationReceived(PriorityValue sender, BindingNotification notification); + + /// + /// Ensures that the current thread is the UI thread. + /// + void VerifyAccess(); } } diff --git a/src/Avalonia.Base/PriorityBindingEntry.cs b/src/Avalonia.Base/PriorityBindingEntry.cs index 25b7bede7e..eb7cf5414c 100644 --- a/src/Avalonia.Base/PriorityBindingEntry.cs +++ b/src/Avalonia.Base/PriorityBindingEntry.cs @@ -92,6 +92,8 @@ namespace Avalonia private void ValueChanged(object value) { + _owner.Owner.Owner?.VerifyAccess(); + var notification = value as BindingNotification; if (notification != null) diff --git a/src/Avalonia.Base/PriorityLevel.cs b/src/Avalonia.Base/PriorityLevel.cs index 122a6df821..b25247deaf 100644 --- a/src/Avalonia.Base/PriorityLevel.cs +++ b/src/Avalonia.Base/PriorityLevel.cs @@ -33,7 +33,6 @@ namespace Avalonia /// internal class PriorityLevel { - private PriorityValue _owner; private object _directValue; private int _nextIndex; @@ -48,13 +47,18 @@ namespace Avalonia { Contract.Requires(owner != null); - _owner = owner; + Owner = owner; Priority = priority; Value = _directValue = AvaloniaProperty.UnsetValue; ActiveBindingIndex = -1; Bindings = new LinkedList(); } + /// + /// Gets the owner of the level. + /// + public PriorityValue Owner { get; } + /// /// Gets the priority of this level. /// @@ -73,7 +77,7 @@ namespace Avalonia set { Value = _directValue = value; - _owner.LevelValueChanged(this); + Owner.LevelValueChanged(this); } } @@ -131,7 +135,7 @@ namespace Avalonia { Value = entry.Value; ActiveBindingIndex = entry.Index; - _owner.LevelValueChanged(this); + Owner.LevelValueChanged(this); } else { @@ -161,7 +165,7 @@ namespace Avalonia /// The error. public void Error(PriorityBindingEntry entry, BindingNotification error) { - _owner.LevelError(this, error); + Owner.LevelError(this, error); } /// @@ -175,14 +179,14 @@ namespace Avalonia { Value = binding.Value; ActiveBindingIndex = binding.Index; - _owner.LevelValueChanged(this); + Owner.LevelValueChanged(this); return; } } Value = DirectValue; ActiveBindingIndex = -1; - _owner.LevelValueChanged(this); + Owner.LevelValueChanged(this); } } } diff --git a/src/Avalonia.Base/PriorityValue.cs b/src/Avalonia.Base/PriorityValue.cs index 3f4b405de9..57e2854014 100644 --- a/src/Avalonia.Base/PriorityValue.cs +++ b/src/Avalonia.Base/PriorityValue.cs @@ -26,7 +26,6 @@ namespace Avalonia /// internal class PriorityValue { - private readonly IPriorityValueOwner _owner; private readonly Type _valueType; private readonly SingleOrDictionary _levels = new SingleOrDictionary(); private object _value; @@ -45,7 +44,7 @@ namespace Avalonia Type valueType, Func validate = null) { - _owner = owner; + Owner = owner; Property = property; _valueType = valueType; _value = AvaloniaProperty.UnsetValue; @@ -53,6 +52,11 @@ namespace Avalonia _validate = validate; } + /// + /// Gets the owner of the value. + /// + public IPriorityValueOwner Owner { get; } + /// /// Gets the property that the value represents. /// @@ -188,9 +192,9 @@ namespace Avalonia Logger.Log( LogEventLevel.Error, LogArea.Binding, - _owner, + Owner, "Error in binding to {Target}.{Property}: {Message}", - _owner, + Owner, Property, error.Error.Message); } @@ -264,19 +268,19 @@ namespace Avalonia if (notification == null || notification.HasValue) { - _owner?.Changed(this, old, _value); + Owner?.Changed(this, old, _value); } if (notification != null) { - _owner?.BindingNotificationReceived(this, notification); + Owner?.BindingNotificationReceived(this, notification); } } else { Logger.Error( LogArea.Binding, - _owner, + Owner, "Binding produced invalid value for {$Property} ({$PropertyType}): {$Value} ({$ValueType})", Property.Name, _valueType, diff --git a/src/Avalonia.Base/Properties/AssemblyInfo.cs b/src/Avalonia.Base/Properties/AssemblyInfo.cs index f57afcdd70..2ee7378413 100644 --- a/src/Avalonia.Base/Properties/AssemblyInfo.cs +++ b/src/Avalonia.Base/Properties/AssemblyInfo.cs @@ -6,4 +6,5 @@ using System.Runtime.CompilerServices; [assembly: AssemblyTitle("Avalonia.Base")] [assembly: InternalsVisibleTo("Avalonia.Base.UnitTests")] +[assembly: InternalsVisibleTo("Avalonia.UnitTests")] [assembly: InternalsVisibleTo("DynamicProxyGenAssembly2")] \ No newline at end of file diff --git a/src/Avalonia.Base/Threading/Dispatcher.cs b/src/Avalonia.Base/Threading/Dispatcher.cs index 6ef55d1735..d46b7142f4 100644 --- a/src/Avalonia.Base/Threading/Dispatcher.cs +++ b/src/Avalonia.Base/Threading/Dispatcher.cs @@ -17,8 +17,8 @@ namespace Avalonia.Threading /// public class Dispatcher : IDispatcher { - private readonly IPlatformThreadingInterface _platform; private readonly JobRunner _jobRunner; + private IPlatformThreadingInterface _platform; public static Dispatcher UIThread { get; } = new Dispatcher(AvaloniaLocator.Current.GetService()); @@ -26,17 +26,25 @@ namespace Avalonia.Threading public Dispatcher(IPlatformThreadingInterface platform) { _platform = platform; - if(_platform == null) - //TODO: Unit test mode, fix that somehow - return; _jobRunner = new JobRunner(platform); - _platform.Signaled += _jobRunner.RunJobs; + + if (_platform != null) + { + _platform.Signaled += _jobRunner.RunJobs; + } } - /// + /// + /// Checks that the current thread is the UI thread. + /// public bool CheckAccess() => _platform?.CurrentThreadIsLoopThread ?? true; - /// + /// + /// Checks that the current thread is the UI thread and throws if not. + /// + /// + /// The current thread is not the UI thread. + /// public void VerifyAccess() { if (!CheckAccess()) @@ -75,5 +83,24 @@ namespace Avalonia.Threading { _jobRunner?.Post(action, priority); } + + /// + /// Allows unit tests to change the platform threading interface. + /// + internal void UpdateServices() + { + if (_platform != null) + { + _platform.Signaled -= _jobRunner.RunJobs; + } + + _platform = AvaloniaLocator.Current.GetService(); + _jobRunner.UpdateServices(); + + if (_platform != null) + { + _platform.Signaled += _jobRunner.RunJobs; + } + } } } \ No newline at end of file diff --git a/src/Avalonia.Base/Threading/JobRunner.cs b/src/Avalonia.Base/Threading/JobRunner.cs index 655ef34288..e212cd9a2e 100644 --- a/src/Avalonia.Base/Threading/JobRunner.cs +++ b/src/Avalonia.Base/Threading/JobRunner.cs @@ -3,7 +3,6 @@ using System; using System.Collections.Generic; -using System.Threading; using System.Threading.Tasks; using Avalonia.Platform; @@ -14,8 +13,8 @@ namespace Avalonia.Threading /// internal class JobRunner { - private readonly IPlatformThreadingInterface _platform; private readonly Queue _queue = new Queue(); + private IPlatformThreadingInterface _platform; public JobRunner(IPlatformThreadingInterface platform) { @@ -82,6 +81,14 @@ namespace Avalonia.Threading AddJob(new Job(action, priority, true)); } + /// + /// Allows unit tests to change the platform threading interface. + /// + internal void UpdateServices() + { + _platform = AvaloniaLocator.Current.GetService(); + } + private void AddJob(Job job) { var needWake = false; @@ -91,7 +98,7 @@ namespace Avalonia.Threading _queue.Enqueue(job); } if (needWake) - _platform.Signal(); + _platform?.Signal(); } /// diff --git a/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Binding.cs b/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Binding.cs index 5e286305d2..cd8184b140 100644 --- a/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Binding.cs +++ b/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Binding.cs @@ -365,7 +365,7 @@ namespace Avalonia.Base.UnitTests } [Fact] - public async void Bind_With_Scheduler_Executes_On_Scheduler() + public async Task Bind_With_Scheduler_Executes_On_Scheduler() { var target = new Class1(); var source = new Subject(); @@ -375,16 +375,16 @@ namespace Avalonia.Base.UnitTests threadingInterfaceMock.SetupGet(mock => mock.CurrentThreadIsLoopThread) .Returns(() => Thread.CurrentThread.ManagedThreadId == currentThreadId); - using (AvaloniaLocator.EnterScope()) - { - AvaloniaLocator.CurrentMutable.Bind().ToConstant(threadingInterfaceMock.Object); - AvaloniaLocator.CurrentMutable.Bind().ToConstant(AvaloniaScheduler.Instance); + var services = new TestServices( + scheduler: AvaloniaScheduler.Instance, + threadingInterface: threadingInterfaceMock.Object); + using (UnitTestApplication.Start(services)) + { target.Bind(Class1.QuxProperty, source); await Task.Run(() => source.OnNext(6.7)); } - } /// diff --git a/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Threading.cs b/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Threading.cs new file mode 100644 index 0000000000..c4c5414e7f --- /dev/null +++ b/tests/Avalonia.Base.UnitTests/AvaloniaObjectTests_Threading.cs @@ -0,0 +1,168 @@ +// Copyright (c) The Avalonia Project. All rights reserved. +// Licensed under the MIT license. See licence.md file in the project root for full license information. + +using System; +using System.Reactive.Subjects; +using System.Threading; +using System.Threading.Tasks; +using Avalonia.Platform; +using Avalonia.UnitTests; +using Xunit; + +namespace Avalonia.Base.UnitTests +{ + public class AvaloniaObjectTests_Threading + { + [Fact] + public void StyledProperty_GetValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.GetValue(Class1.StyledProperty)); + } + } + + [Fact] + public void StyledProperty_SetValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.SetValue(Class1.StyledProperty, "foo")); + } + } + + [Fact] + public void Setting_StyledProperty_Binding_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => + target.Bind( + Class1.StyledProperty, + new BehaviorSubject("foo"))); + } + } + + [Fact] + public void StyledProperty_ClearValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.ClearValue(Class1.StyledProperty)); + } + } + + [Fact] + public void StyledProperty_IsSet_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.IsSet(Class1.StyledProperty)); + } + } + + [Fact] + public void DirectProperty_GetValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.GetValue(Class1.DirectProperty)); + } + } + + [Fact] + public void DirectProperty_SetValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.SetValue(Class1.DirectProperty, "foo")); + } + } + + [Fact] + public void Setting_DirectProperty_Binding_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => + target.Bind( + Class1.DirectProperty, + new BehaviorSubject("foo"))); + } + } + + [Fact] + public void DirectProperty_ClearValue_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.ClearValue(Class1.DirectProperty)); + } + } + + [Fact] + public void DirectProperty_IsSet_Should_Throw() + { + using (UnitTestApplication.Start(new TestServices(threadingInterface: new ThreadingInterface()))) + { + var target = new Class1(); + + Assert.Throws(() => target.IsSet(Class1.DirectProperty)); + } + } + + private class Class1 : AvaloniaObject + { + public static readonly StyledProperty StyledProperty = + AvaloniaProperty.Register("Foo", "foodefault"); + + public static readonly DirectProperty DirectProperty = + AvaloniaProperty.RegisterDirect("Qux", _ => null, (o, v) => { }); + } + + private class ThreadingInterface : IPlatformThreadingInterface + { + public ThreadingInterface(bool isLoopThread = false) + { + CurrentThreadIsLoopThread = isLoopThread; + } + + public bool CurrentThreadIsLoopThread { get; set; } + + public event Action Signaled; + + public void RunLoop(CancellationToken cancellationToken) + { + throw new NotImplementedException(); + } + + public void Signal() + { + throw new NotImplementedException(); + } + + public IDisposable StartTimer(TimeSpan interval, Action tick) + { + throw new NotImplementedException(); + } + } + } +} diff --git a/tests/Avalonia.UnitTests/TestServices.cs b/tests/Avalonia.UnitTests/TestServices.cs index 27f19e09a8..c976438d1e 100644 --- a/tests/Avalonia.UnitTests/TestServices.cs +++ b/tests/Avalonia.UnitTests/TestServices.cs @@ -12,6 +12,7 @@ using Avalonia.Shared.PlatformSupport; using Avalonia.Styling; using Avalonia.Themes.Default; using Avalonia.Rendering; +using System.Reactive.Concurrency; namespace Avalonia.UnitTests { @@ -70,6 +71,7 @@ namespace Avalonia.UnitTests Func renderer = null, IPlatformRenderInterface renderInterface = null, IRenderLoop renderLoop = null, + IScheduler scheduler = null, IStandardCursorFactory standardCursorFactory = null, IStyler styler = null, Func theme = null, @@ -86,6 +88,7 @@ namespace Avalonia.UnitTests Renderer = renderer; RenderInterface = renderInterface; RenderLoop = renderLoop; + Scheduler = scheduler; StandardCursorFactory = standardCursorFactory; Styler = styler; Theme = theme; @@ -103,6 +106,7 @@ namespace Avalonia.UnitTests public Func Renderer { get; } public IPlatformRenderInterface RenderInterface { get; } public IRenderLoop RenderLoop { get; } + public IScheduler Scheduler { get; } public IStandardCursorFactory StandardCursorFactory { get; } public IStyler Styler { get; } public Func Theme { get; } @@ -120,6 +124,7 @@ namespace Avalonia.UnitTests Func renderer = null, IPlatformRenderInterface renderInterface = null, IRenderLoop renderLoop = null, + IScheduler scheduler = null, IStandardCursorFactory standardCursorFactory = null, IStyler styler = null, Func theme = null, @@ -138,6 +143,7 @@ namespace Avalonia.UnitTests renderer: renderer ?? Renderer, renderInterface: renderInterface ?? RenderInterface, renderLoop: renderLoop ?? RenderLoop, + scheduler: scheduler ?? Scheduler, standardCursorFactory: standardCursorFactory ?? StandardCursorFactory, styler: styler ?? Styler, theme: theme ?? Theme, diff --git a/tests/Avalonia.UnitTests/UnitTestApplication.cs b/tests/Avalonia.UnitTests/UnitTestApplication.cs index 8167263e3d..d40a382bca 100644 --- a/tests/Avalonia.UnitTests/UnitTestApplication.cs +++ b/tests/Avalonia.UnitTests/UnitTestApplication.cs @@ -8,6 +8,9 @@ using Avalonia.Platform; using Avalonia.Styling; using Avalonia.Controls; using Avalonia.Rendering; +using Avalonia.Threading; +using System.Reactive.Disposables; +using System.Reactive.Concurrency; namespace Avalonia.UnitTests { @@ -30,7 +33,12 @@ namespace Avalonia.UnitTests var scope = AvaloniaLocator.EnterScope(); var app = new UnitTestApplication(services); AvaloniaLocator.CurrentMutable.BindToSelf(app); - return scope; + Dispatcher.UIThread.UpdateServices(); + return Disposable.Create(() => + { + scope.Dispose(); + Dispatcher.UIThread.UpdateServices(); + }); } public override void RegisterServices() @@ -48,6 +56,7 @@ namespace Avalonia.UnitTests .Bind().ToConstant(Services.RenderInterface) .Bind().ToConstant(Services.RenderLoop) .Bind().ToConstant(Services.ThreadingInterface) + .Bind().ToConstant(Services.Scheduler) .Bind().ToConstant(Services.StandardCursorFactory) .Bind().ToConstant(Services.Styler) .Bind().ToConstant(Services.WindowingPlatform)