FIX: Point terminal layer getter function

[SMoraisAnsys]

Duplicates PR #945 which was created from a fork.

---

The getter function for layer of the point terminal always returned the top layer name irrespective of where the terminal was located. This has been fixed.
The getter function of location has also been refactored to simply implementation.

Fixes #944

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 55.55556% with 4 lines in your changes missing coverage. Please review.

Project coverage is 82.66%. Comparing base (fc0c8fa) to head (7a68a8a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #965      +/-   ##
==========================================
- Coverage   82.68%   82.66%   -0.02%     
==========================================
  Files         160      160              
  Lines       21098    21100       +2     
==========================================
- Hits        17444    17443       -1     
- Misses       3654     3657       +3     

[SMoraisAnsys]

@skandak-ansys We had a problem with the VM used in CICD. Now things should be working again :)

[svandenb-dev]

LGTM

[hui-zhou-a]

@svandenb-dev the layer property should be in PointTerminal class not Terminal

[skandak-ansys]

@hui-zhou-a layer property is available for both PointTerminal and PadstackInstanceTerminal. So I added a try-except to capture terminals where layer property is invalid and left it inside Terminal class, where it initially was.

[hui-zhou-a]

@skandak-ansys logger.error should be replaced by Raise RuntimeError. Logging the error will hide the problem.

[hui-zhou-a]

@skandak-ansys GetParameter should only be called in PointTerminal and PadstackInstanceTermal class. This is a code structural problem. I will fix it in another PR. Thanks for your contribution!