Render before adding to the splitView

[aaomidi]

This PR fixes #10213

It looks like the adding to splitView depends on this.body of the AccountPanel to not be undefined. This field gets populated in the render() method:

azuredatastudio/src/vs/base/browser/ui/splitview/paneview.ts

Line 229 in 93f35ca

this.body = append(this.element, $('.pane-body'));

So calling render before adding to the splitView should fix the problem.

[coveralls]

Coverage remained the same at 32.731% when pulling bd0f300 on amir/fix/azurePane into 3e5b7fd on master.

[aaomidi]

I'm not super familiar with the splitView code, is this the correct solution or is this fixing a symptom caused by somewhere else?

[ranasaria]

I have verified that the fix resolves the issue.

[ranasaria]

I'm not super familiar with the splitView code, is this the correct solution or is this fixing a symptom caused by somewhere else?

The change is simple enough that if there is a better fix we can do it later.

[ranasaria]

The fix looks good to me. As long as we understand why this was not needed before we should be good.