ionic-team / ionic-team/ionic-framework

bug: karma tests sometimes warn about missing contentId on menu even if present

Đang mở
#24,566 2 bình luận 0 reaction 0 người được giao Xem trên GitHub
package: angular type: bug
Ngôn ngữ chính
TypeScript
Star
52.7k
Fork
13.3k
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
51

Mô tả

### Prerequisites

- [X] I have read the [Contributing Guidelines](https://github.com/ionic-team/ionic-framework/blob/main/.github/CONTRIBUTING.md#creating-an-issue).
- [X] I agree to follow the [Code of Conduct](https://ionicframework.com/code-of-conduct).
- [X] I have searched for [existing issues](https://github.com/ionic-team/ionic-framework/issues) that already report this problem, without success.

### Ionic Framework Version

- [ ] v4.x
- [X] v5.x
- [X] v6.x

### Current Behavior

Sometimes when running Karma tests in an Ionic Angular side menu app, Ionic will give the following errors/warnings:

```
ERROR: 'Menu: must have a "content" element to listen for drag events on.'

WARN: 'split pane does not have a specified main node'
```

This is happening even if `contentId` is set on `ion-split-pane` and `ion-menu`. Digging into it, it appears that `contentId` sometimes does not work but `content-id` does. Looks to be some sort of race condition, possibly with either our Angular component wrappers or Web Component upgrading.

### Expected Behavior

I would expect no error/warning to be logged when using contentId as directed.

### Steps to Reproduce

1. Run `ionic start foo sidemenu --type=angular`
2. Run `cd ./foo`
3. Run `ng test`
4. Observe that an error/warning is logged. Since this is a race condition, you may need to run this a few times.

### Code Reproduction URL

_No response_

### Ionic Info

_No response_

### Additional Information

_No response_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.