zowe / zowe-explorer-vscode

Visual Studio Code Extension for Zowe, which lets users interact with z/OS Data Sets, Unix System Services, and Jobs on a remote mainframe instance. Powered by Zowe SDKs.
Eclipse Public License 2.0
159 stars 89 forks source link

Regression on: fix(ProfilesCache): Clear object caches before refresh #2795 #2910

Closed FrankSu1996 closed 1 month ago

FrankSu1996 commented 1 month ago

Describe the bug

Issue #2795 seems to have regressed. When the fix came out in v2.15.2, I tested it and it worked. Today, I stumbled across the behavior again, where removing the single instance of our profile type still caused the ProfilesCache to hold onto the stale reference. The only other thing that has changed in my VsCode environment that I could think of is the zowe-explorer version and VsCode version.

To Reproduce

Steps to reproduce the behavior:

  1. Contribute new profile type using zowe-explorer-api.
  2. Create a team config file. Doesn't matter if it's global/project
  3. Delete the single instance of the contributed profile, and call ProfilesCache.refresh()

Expected behavior

ProfilesCache should correctly remove the profile. But it doesn't.

Screenshots

Desktop (please complete the following information):

github-actions[bot] commented 1 month ago

Thank you for creating a bug report. We will investigate the bug and evaluate its impact on the product. If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

t1m0thyj commented 1 month ago

This may be related to #2888

JillieBeanSim commented 1 month ago

closing with merge of attached PR