-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FOUNDATIONS: Assembly Service #6
Conversation
STX.SPAL.Core/Services/Foundations/Assemblies/AssemblyService.Exceptions.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@LBoullosa @cjdutoit LGTM
As to what Christo mentioned in Discord do we need a couple of test cases for exceptions like DirectoryNotFound and FileNotFound.
@glhays FileNotFoundException is thrown by Assembly.Load, so yes, you are right we need to care about it. |
…tionOccurs -> FAIL
…tionOccurs -> FAIL
…tionOccurs -> FAIL
…tionOccurs -> PASS
.../Services/Foundations/Assemblies/AssemblyServiceTests.Logic.GetApplicationPathsAssemblies.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a advantageous reason to convert the Assembly broker to asynchronous as to not block the constructor while the IO operations are being run?
Closes: #5