Closed Mike-E-angelo closed 4 years ago
What setting are you looking for? The only settings that are being added are regarding navigation patterns. And they should appear on Unit Testing General.
On Thu 9. Jan 2020 at 12:17, Mike-EEE notifications@github.com wrote:
Using this commit:
https://github.com/DragonSpark/Framework/tree/fbcea5605a9f884276d73dc037924e97c4dea683
And in this directory:
https://github.com/DragonSpark/Framework/tree/fbcea5605a9f884276d73dc037924e97c4dea683/DragonSpark
I create a simple class called Class1. I get the dialog:
So far, so good.
However, when accepting and clicking Yes, the test file does not seem to create in the specified namespace/folder, or at all.
I know that in #16 https://github.com/matkoch/resharper-testlinker/issues/16 it was mentioned options were merged, but I am not seeing anything obvious... is there another setting somewhere I should be considering, perhaps?
โ You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7V3RYYBC5K6ENST6XTQ44BVBA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IFA4PPQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7TXLUU47PQCJTZSHPTQ44BVBANCNFSM4KEWUNXQ .
I'm looking for the setting that will create the test as the dialog suggests it will do. :)
FWIW if the file already exists I navigate to it as expected. It's basically the creating of files that is not working.
Well thereโs certainly no setting to enable or disable bugs ๐ค
On Thu 9. Jan 2020 at 13:34, Mike-EEE notifications@github.com wrote:
I'm looking for the setting that will create the test as the dialog suggests it will do. :)
FWIW if the file already exists I navigate to it as expected. It's basically the creating of files that is not working.
โ You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7VOSBTF77WLM5VKB2TQ44KWLA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIQE2ZA#issuecomment-572542308, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7TMMBXXG7YJQFOZC2DQ44KWLANCNFSM4KEWUNXQ .
Well thereโs certainly no setting to enable or disable bugs
YET. ๐
Is this project considered broken, then? ๐ It's obviously frustrating that it worked perfectly fine in previous versions but because of the upgrade path ReSharper forces extensions, it has somehow introduced a breaking change here.
Kind reminder to vote:
@Mike-EEE I'm sorry. This has a much lower priority than anything else, and I haven't had any time to look into that :( I will try this week, but without any promises :/
No worries @matkoch I'm trying to balance my nags with perspective. ๐
Good thing that the rest of R# is so ridiculously awesome or else this would be a bigger deal than it is. ๐
This seems to have actually progressed somehow. I have created a simple repo here:
https://github.com/Mike-EEE/TestLinker.Basic.git
Opening this file and invoking ReSharper_GotoLinkedTypesAction
results in the following dialog for me:
Here are my options:
Please let me know if you are able to reproduce this and/or if you require further logging on my side AND/OR if there is something completely obvious here I am overlooking. ๐
An empty repo?
On Sun 16. Feb 2020 at 11:10, Mike-EEE notifications@github.com wrote:
This seems to have actually progressed somehow. I have created a simple repo here:
https://github.com/Mike-EEE/TestLinker.Basic.git
Opening this file and invoking ReSharper_GotoLinkedTypesAction results in the following dialog for me:
Here are my options:
Please let me know if you are able to reproduce this and/or if you require further logging on my side.
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7QKDA2RQS25BSGKR3LRDEGI5A5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL4CREY#issuecomment-586688659, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7WGREGSXCDQRAESYH3RDEGI5ANCNFSM4KEWUNXQ .
๐ญ๐ญ๐ญ
Hey, you all aren't the only ones that are buggy.
๐๐๐
Try now.
๐๐๐
Just looking at the code, this is completely natural to me. There is no matching pair of implementation/test in the solution. So it canโt infer where to put tests.
On Mon 17. Feb 2020 at 08:55, Mike-EEE notifications@github.com wrote:
๐ญ๐ญ๐ญ
Hey, you all aren't the only ones that are buggy.
๐๐๐
Try now.
๐๐๐
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7QFZAES46VHUB3DGHLRDI7GTA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5MSLY#issuecomment-586860847, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7SOTPVSVLMR4ZBF55DRDI7GTANCNFSM4KEWUNXQ .
Alright, I have stepped into the Twilight Zone here. Why is TestLinker.Testing
not considered?
Iโm talking about a pair of implementation and test classes. This seemed to be the most efficient approach.
On Mon 17. Feb 2020 at 09:12, Mike-EEE notifications@github.com wrote:
Alright, I have stepped into the Twilight Zone here. Why is TestLinker.Testing not considered?
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7RUVZ5SAQLZSQICVKTRDJBGJA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5OAMA#issuecomment-586866736, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7SFGTWTA722WHTW5PLRDJBGJANCNFSM4KEWUNXQ .
You've lost me, unfortunately. From the top. ๐ I am attempting to create a test class from utilizing ReSharper_GotoLinkedTypesAction
in a code file (Class1.cs
) which historically finds the test project (TestLinker.Testing
) and creates a test class (Class1Tests.cs
). It is now producing the dialog presented above instead. What do I have misunderstood here?
The plugin doesnโt work if there is not at least a single known pair of implementation and test class. This is the case in the repository youโve created. Add, for instance, Class.cs and ClassTests.cs with according classes, and it would work for the next pair.
On Mon 17. Feb 2020 at 09:21, Mike-EEE notifications@github.com wrote:
You've lost me, unfortunately. From the top. ๐ I am attempting to create a test class from utilizing ReSharper_GotoLinkedTypesAction in a code file (Class1.cs) which historically finds the test project ( TestLinker.Testing) and creates a test class (Class1Tests.cs). It is now producing the dialog presented above instead. What do I have misunderstood here?
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7VGD3UJB6DMPKQV5OLRDJCIZA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5O2UI#issuecomment-586870097, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7W6TFGEUIXNGLRMHGTRDJCIZANCNFSM4KEWUNXQ .
This is expected behavior. It always worked like that.
On Mon 17. Feb 2020 at 09:23, Matthias ithrowexceptions@gmail.com wrote:
The plugin doesnโt work if there is not at least a single known pair of implementation and test class. This is the case in the repository youโve created. Add, for instance, Class.cs and ClassTests.cs with according classes, and it would work for the next pair.
On Mon 17. Feb 2020 at 09:21, Mike-EEE notifications@github.com wrote:
You've lost me, unfortunately. From the top. ๐ I am attempting to create a test class from utilizing ReSharper_GotoLinkedTypesAction in a code file (Class1.cs) which historically finds the test project ( TestLinker.Testing) and creates a test class (Class1Tests.cs). It is now producing the dialog presented above instead. What do I have misunderstood here?
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7VGD3UJB6DMPKQV5OLRDJCIZA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5O2UI#issuecomment-586870097, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7W6TFGEUIXNGLRMHGTRDJCIZANCNFSM4KEWUNXQ .
OK! I am with you now. Previously I have always had at least one "paired connection" so that I never had to create one, so that explains that.
Now, for this solution, if you link the Class1
by creating a Class1Tests
in TestLinker.Testing
, and then create a Class2
, invoke ReSharper_GotoLinkedTypesAction
on Class2
does it create Class2Tests
? It does not for me and I can get some logs for you if needed.
That should be working like that, yes.
On Mon 17. Feb 2020 at 09:26, Mike-EEE notifications@github.com wrote:
OK! I am with you now. Previously I have always had at least one "paired connection" so that I never had to create one, so that explains that.
Now, for this solution, if you link the Class1 by creating a Class1Tests in TestLinker.Testing, and then create a Class2, invoke ReSharper_GotoLinkedTypesAction on Class2 does it create Class2Tests ? It does not for me and I can get some logs for you if needed.
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7SVFWVGE4VLA3YAOSDRDJC4PA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5PKFQ#issuecomment-586872086, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7VYSZYC3S6YZQYVDEDRDJC4PANCNFSM4KEWUNXQ .
I know it should be working like that. ๐ But I am asking you if it does work like that on your side.
FWIW I added it here to try:
https://github.com/Mike-EEE/TestLinker.Basic/blob/master/TestLinker/Class2.cs
Capture of what I am seeing I PROMISE I AM NOT MAKING THIS UP. ๐
Alright, I've created logs and sent it to you JB email account. Please let me know if you require any further/additional information to further diagnose this issue. ๐
Have you started devenv.exe with /Resharper.Internal? Any exception shown in the statusbar?
On Mon 17. Feb 2020 at 09:49, Mike-EEE notifications@github.com wrote:
Alright, I've created logs and sent it to you JB email account. Please let me know if you require any further/additional information to further diagnose this issue. ๐
โ You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/matkoch/resharper-testlinker/issues/17?email_source=notifications&email_token=ABGGB7R3VP4FD4FEYZEX7G3RDJFTLA5CNFSM4KEWUNX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5RN7Q#issuecomment-586880766, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGGB7US3RA3HYWTWTCTBVLRDJFTLANCNFSM4KEWUNXQ .
I did and sent you the logs per above... did you receive them?
Are you saying that you cannot reproduce this on your side with the provided SLN?
Ah apologies @matkoch ... details. The configuration I had was for logging only and not /Resharper.Internal
. I now have two ReSharper Visual Studio shortcuts on my desktop. :)
This is what is produced:
This action cannot be executed on the UiTaskExecutorWorker:114 thread. It's only allowed to run on the :1 thread.
--- EXCEPTION #1/2 [ThreadAccessException]
Message = โThis action cannot be executed on the UiTaskExecutorWorker:114 thread. It's only allowed to run on the :1 thread.โ
ExceptionPath = Root.InnerException
ClassName = JetBrains.Threading.ThreadAccessException
HResult = COR_E_APPLICATION=80131600
Source = JetBrains.Platform.Core
StackTraceString = โ
at JetBrains.Threading.JetDispatcher.AssertAccess()
at JetBrains.DocumentModel.Transactions.DocumentTransactionManager.StartTransaction(String commandName, Boolean optimizeChangesOnCommit)
at JetBrains.DocumentModel.Transactions.DocumentTransactionCookie..ctor(DocumentTransactionManager manager, DefaultAction defaultAction, String commandName, Boolean optimizeChangesOnCommit, IUndoRedoHandler undoRedoHandler, IProgressIndicator pi)
at JetBrains.DocumentManagers.Transactions.ProjectModelTransactionFactory.CreateTransactionCookie(DefaultAction defaultAction, String commandName, IUndoRedoHandler handler, Boolean optimizeChangesOnCommit, IProgressIndicator pi)
at JetBrains.DocumentManagers.impl.ProjectFileUtil.GetOrCreateProjectFolder(IProjectFolder baseFolder, FileSystemPath path, IProjectModelTransactionCookie cookie, IFolderCreationContext context)
at ReSharperPlugin.TestLinker.Utils.ModificationUtility.GetLinkedTypeFolder(String linkedTypeNamespace, IProject linkedTypeProject)
at ReSharperPlugin.TestLinker.Utils.ModificationUtility.TryCreateTestOrProductionClass(ITypeElement sourceType, ITextControl textControl)
at ReSharperPlugin.TestLinker.Navigation.LinkedTypesSearchRequest.Search(IProgressIndicator progressIndicator)
at JetBrains.ReSharper.Feature.Services.Navigation.Requests.SearchRequest.<>c__DisplayClass9_1.<Search>b__1()
at JetBrains.Application.Threading.IShellLocksEx.ExecuteWithReadLock(IShellLocks thัs, Action F)
at JetBrains.ReSharper.Feature.Services.Navigation.Requests.SearchRequest.<>c__DisplayClass9_0.<Search>b__0(IProgressIndicator pro)
at JetBrains.UI.SrcView.Progress.UITaskExecutorRun.RunStartTaskCallTaskAction(IProperty`1 isDone, ProgressIndicator progressIndicator, TaskThreadAffinity taskThreadAffinity, Action`1 FTask, JetDispatcher dispatcherPrimary)
โ
--- Outer ---
--- EXCEPTION #2/2 [LoggerException]
Message = โThis action cannot be executed on the UiTaskExecutorWorker:114 thread. It's only allowed to run on the :1 thread.โ
ExceptionPath = Root
ClassName = JetBrains.Util.LoggerException
Data.ManagedThreadName = UiTaskExecutorWorker
Data.LastExtension = cs
Data.SccRevisionShell = โ<there are no packages matching the criteria>โ
Data.HostProductInfo = โJetBrains ReSharper Ultimate 2019.3.2 Build 193.0.20200209.70108โ
Data.SubProducts.#0 = โJetBrains dotCover 2019.3.2 Build 2019.3.20200209.94935โ
Data.SubProducts.#1 = โJetBrains dotTrace 2019.3.2 Build 2019.3.20200209.94927โ
Data.SubProducts.#2 = โJetBrains ReSharper 2019.3.2 Build 2019.3.20200209.72252โ
Data.SubProducts.#3 = โJetBrains dotMemory 2019.3.2 Build 2019.3.20200209.94942โ
Data.SccRevisionEnv = โ
Platform\Core\Shell,
Platform\VisualStudio:
git::refs/heads/net193-eap8-rtm-2019.3.2
โ
Data.VsVersion = 16.4.29806.167
InnerException = โException #1 at Root.InnerExceptionโ
HResult = COR_E_APPLICATION=80131600
StackTraceString = โ
at JetBrains.UI.SrcView.Progress.UITaskExecutorRun.RunStartTaskCallTaskAction(IProperty`1 isDone, ProgressIndicator progressIndicator, TaskThreadAffinity taskThreadAffinity, Action`1 FTask, JetDispatcher dispatcherPrimary)
at JetBrains.UI.SrcView.Progress.UITaskExecutorRun.<>c__DisplayClass11_0.<RunStartTask>b__0()
at JetBrains.Threading.JetDispatcher.Closure.Execute()
at JetBrains.Util.Concurrency.WinJetDispatcher.ProcessQueue(Int32 nMinBucket)
at JetBrains.Util.Concurrency.WinJetDispatcher.<>c__DisplayClass18_0.<BeginInvokeCore>b__0()
at System.Windows.Threading.DispatcherOperation.InvokeDelegateCore()
at System.Windows.Threading.DispatcherOperation.InvokeImpl()
at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(Object state)
at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at MS.Internal.CulturePreservingExecutionContext.Run(CulturePreservingExecutionContext executionContext, ContextCallback callback, Object state)
at System.Windows.Threading.DispatcherOperation.Invoke()
at System.Windows.Threading.Dispatcher.ProcessQueue()
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
at JetBrains.Interop.WinApi.User32Dll.DispatchMessageW(MSG* lpmsg)
at JetBrains.Util.Concurrency.WinJetDispatcher.WinJetDispatcherStaticMethods.InternalPumpMessagesOnce()
at JetBrains.Util.Concurrency.WinJetDispatcher.WinJetDispatcherStaticMethods.JetBrains.Util.Concurrency.IJetDispatcherStaticMethods.Run()
at JetBrains.Threading.JetDispatcher.<>c__DisplayClass22_1.<CreateDispatcherThread>b__1()
at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()
โ
Also reported here: https://youtrack.jetbrains.com/issue/DEXP-499061
Now that we got that sorted. ๐ What are your thoughts on this @matkoch? Big deal or nah? ๐
I just installed 2019.3.3
with the slight hope that perhaps something somewhere was adjusted somehow and someway to address this issue but the problem still happens. ๐ญ๐ญ๐ญ
Keep in mind that I get a not-so-subtle reminder of this issue whenever I create a new class and want to create a test for it so I am passing the savings (read: pain ๐) onto you. ๐๐๐
And I am currently full throttle into designing new classes/modelling for my project right now.
๐๐๐
Kind reminder to vote for this issue here:
https://youtrack.jetbrains.com/issue/RSRP-459966
(Thinking it would be quite the accomplishment to nag JB so much on this issue their own developers start voting for it. ๐)
This looks to be working now in the private link you sent me @matkoch. THANK YOU SO MUCH! It would be great to get this posted to a preview feed or in the actual public release feed itself. Thank you once again for your efforts and patience in fixing this issue!
In case anyone else is interested in these links until they are posted in an official capacity on a feed:
2019.3.2
+ version:
https://www.dropbox.com/s/dn8m3c3t8aliirg/ReSharper.TestLinker.2019.3.2.nupkg?dl=0
2020.1 EAP
version:
https://www.dropbox.com/s/t6gfavbcgkwqyl3/ReSharper.TestLinker.2020.1.0.nupkg?dl=0
Using this commit:
https://github.com/DragonSpark/Framework/tree/fbcea5605a9f884276d73dc037924e97c4dea683
And in this directory:
https://github.com/DragonSpark/Framework/tree/fbcea5605a9f884276d73dc037924e97c4dea683/DragonSpark
I create a simple class called
Class1
. I get the dialog:So far, so good.
However, when accepting and clicking
Yes
, the test file does not seem to create in the specified namespace/folder, or at all.I know that in #16 it was mentioned options were merged, but I am not seeing anything obvious... is there another setting somewhere I should be considering, perhaps?